Skip to content

Commit

Permalink
Don't require authentication to find gems by owner
Browse files Browse the repository at this point in the history
Closes #359.
  • Loading branch information
sferik committed Oct 7, 2011
1 parent 72fdbe1 commit 631316a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
9 changes: 3 additions & 6 deletions app/controllers/api/v1/owners_controller.rb
@@ -1,12 +1,9 @@
class Api::V1::OwnersController < Api::BaseController class Api::V1::OwnersController < Api::BaseController

skip_before_filter :verify_authenticity_token, :only => [:create, :destroy] skip_before_filter :verify_authenticity_token, :only => [:create, :destroy]

before_filter :authenticate_with_api_key, :except => :show
before_filter :authenticate_with_api_key before_filter :verify_authenticated_user, :except => :show
before_filter :verify_authenticated_user
before_filter :find_rubygem before_filter :find_rubygem
before_filter :verify_gem_ownership before_filter :verify_gem_ownership, :except => :show

respond_to :yaml, :xml, :json, :only => :show respond_to :yaml, :xml, :json, :only => :show


def show def show
Expand Down
1 change: 0 additions & 1 deletion features/owner.feature
Expand Up @@ -81,6 +81,5 @@ Feature: Manage owners


Examples: Examples:
| action | | action |
| list the owners of gem "OGem" |
| add the owner "new@owner.org" to the rubygem "OGem" | | add the owner "new@owner.org" to the rubygem "OGem" |
| remove the owner "original@owner.org" from the rubygem "OGem" | | remove the owner "original@owner.org" from the rubygem "OGem" |

0 comments on commit 631316a

Please sign in to comment.