[ARVADOS] updated: 11fc08947a6630ef36fee3819eec1272a22997da

Git user git at public.curoverse.com
Mon Mar 14 10:39:40 EDT 2016


Summary of changes:
 .../app/controllers/arvados/v1/api_client_authorizations_controller.rb  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

  discards  e057c658850fad7612472d28df4d428fed61f301 (commit)
       via  11fc08947a6630ef36fee3819eec1272a22997da (commit)

This update added new revisions after undoing existing revisions.  That is
to say, the old revision is not a strict subset of the new revision.  This
situation occurs when you --force push a change and generate a repository
containing something like this:

 * -- * -- B -- O -- O -- O (e057c658850fad7612472d28df4d428fed61f301)
            \
             N -- N -- N (11fc08947a6630ef36fee3819eec1272a22997da)

When this happens we assume that you've already had alert emails for all
of the O revisions, and so we here report only the revisions in the N
branch from the common base, B.

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 11fc08947a6630ef36fee3819eec1272a22997da
Author: Tom Clegg <tom at curoverse.com>
Date:   Mon Mar 14 10:39:31 2016 -0400

    8079: Allow where(api_token: foo) and disallow where(api_token: ["contains", "f"])

diff --git a/services/api/app/controllers/arvados/v1/api_client_authorizations_controller.rb b/services/api/app/controllers/arvados/v1/api_client_authorizations_controller.rb
index 0040ee0..454ce10 100644
--- a/services/api/app/controllers/arvados/v1/api_client_authorizations_controller.rb
+++ b/services/api/app/controllers/arvados/v1/api_client_authorizations_controller.rb
@@ -61,7 +61,9 @@ class Arvados::V1::ApiClientAuthorizationsController < ApplicationController
     end
     if @where
       wanted_scopes << @where['scopes']
-      @where.select! { |attr, val| attr == 'uuid' }
+      @where.select! { |attr, val|
+        val.is_a?(String) && (attr == 'uuid' || attr == 'api_token')
+      }
     end
     @objects = model_class.
       includes(:user, :api_client).

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list