[ARVADOS] updated: ff3686b372cd598ee990acd990af2b04daf91ada
git at public.curoverse.com
git at public.curoverse.com
Fri May 30 15:18:50 EDT 2014
Summary of changes:
doc/api/methods/users.html.textile.liquid | 4 ---
sdk/cli/bin/arv | 1 -
.../app/controllers/arvados/v1/users_controller.rb | 37 ++--------------------
services/api/config/routes.rb | 1 -
4 files changed, 2 insertions(+), 41 deletions(-)
via ff3686b372cd598ee990acd990af2b04daf91ada (commit)
from b01252252e30851993c25817e07b574be66ce17f (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit ff3686b372cd598ee990acd990af2b04daf91ada
Author: radhika <radhika at curoverse.com>
Date: Fri May 30 15:17:54 2014 -0400
2895: remove users.event_stream api
diff --git a/doc/api/methods/users.html.textile.liquid b/doc/api/methods/users.html.textile.liquid
index 59fa856..33f884b 100644
--- a/doc/api/methods/users.html.textile.liquid
+++ b/doc/api/methods/users.html.textile.liquid
@@ -42,10 +42,6 @@ table(table table-bordered table-condensed).
|_. Argument |_. Type |_. Description |_. Location |_. Example |
{background:#ccffcc}.|uuid|string|The UUID of the User in question.|path||
-h2. event_stream
-
-event_stream users
-
Arguments:
table(table table-bordered table-condensed).
diff --git a/sdk/cli/bin/arv b/sdk/cli/bin/arv
index d047204..53956aa 100755
--- a/sdk/cli/bin/arv
+++ b/sdk/cli/bin/arv
@@ -313,7 +313,6 @@ end
case api_method
when
- 'arvados.users.event_stream',
'arvados.jobs.log_stream',
'arvados.jobs.log_tail_follow'
diff --git a/services/api/app/controllers/arvados/v1/users_controller.rb b/services/api/app/controllers/arvados/v1/users_controller.rb
index c2a32f0..781bd88 100644
--- a/services/api/app/controllers/arvados/v1/users_controller.rb
+++ b/services/api/app/controllers/arvados/v1/users_controller.rb
@@ -1,8 +1,8 @@
class Arvados::V1::UsersController < ApplicationController
skip_before_filter :find_object_by_uuid, only:
- [:activate, :event_stream, :current, :system, :setup]
+ [:activate, :current, :system, :setup]
skip_before_filter :render_404_if_no_object, only:
- [:activate, :event_stream, :current, :system, :setup]
+ [:activate, :current, :system, :setup]
before_filter :admin_required, only: [:setup, :unsetup]
def current
@@ -14,39 +14,6 @@ class Arvados::V1::UsersController < ApplicationController
show
end
- class ChannelStreamer
- Q_UPDATE_INTERVAL = 12
- def initialize(opts={})
- @opts = opts
- end
- def each
- return unless @opts[:channel]
- @redis = Redis.new(:timeout => 0)
- @redis.subscribe(@opts[:channel]) do |event|
- event.message do |channel, msg|
- yield msg + "\n"
- end
- end
- end
- end
-
- def event_stream
- channel = current_user.andand.uuid
- if current_user.andand.is_admin
- channel = params[:uuid] || channel
- end
- if client_accepts_plain_text_stream
- self.response.headers['Last-Modified'] = Time.now.ctime.to_s
- self.response_body = ChannelStreamer.new(channel: channel)
- else
- render json: {
- href: url_for(uuid: channel),
- comment: ('To retrieve the event stream as plain text, ' +
- 'use a request header like "Accept: text/plain"')
- }
- end
- end
-
def activate
if current_user.andand.is_admin && params[:uuid]
@object = User.find params[:uuid]
diff --git a/services/api/config/routes.rb b/services/api/config/routes.rb
index 0223c04..49d78e4 100644
--- a/services/api/config/routes.rb
+++ b/services/api/config/routes.rb
@@ -49,7 +49,6 @@ Server::Application.routes.draw do
resources :users do
get 'current', on: :collection
get 'system', on: :collection
- get 'event_stream', on: :member
post 'activate', on: :member
post 'setup', on: :collection
post 'unsetup', on: :member
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list