diff options
author | Robert Schilling <rschilling@student.tugraz.at> | 2017-01-23 19:06:57 +0300 |
---|---|---|
committer | Robert Schilling <rschilling@student.tugraz.at> | 2017-01-23 19:06:57 +0300 |
commit | 18417c9ee8b0e0bfa4ee41dd8ada969c0ede07f9 (patch) | |
tree | 3f7811eb4f5b005fcac7e6fdfb4cf0120ae9f2bb /lib | |
parent | 6c65f7a7ce52fadcf53ea2395103de32b0107874 (diff) |
Grapify all endpoints of the deploy keys API
Diffstat (limited to 'lib')
-rw-r--r-- | lib/api/deploy_keys.rb | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/lib/api/deploy_keys.rb b/lib/api/deploy_keys.rb index 85360730841..28eea79fbaf 100644 --- a/lib/api/deploy_keys.rb +++ b/lib/api/deploy_keys.rb @@ -38,26 +38,25 @@ module API present key, with: Entities::SSHKey end - # TODO: for 9.0 we should check if params are there with the params block - # grape provides, at this point we'd change behaviour so we can't - # Behaviour now if you don't provide all required params: it renders a - # validation error or two. desc 'Add new deploy key to currently authenticated user' do success Entities::SSHKey end + params do + requires :key, type: String, desc: 'The new deploy key' + requires :title, type: String, desc: 'The name of the deploy key' + end post ":id/#{path}" do - attrs = attributes_for_keys [:title, :key] - attrs[:key].strip! if attrs[:key] + params[:key].strip! # Check for an existing key joined to this project - key = user_project.deploy_keys.find_by(key: attrs[:key]) + key = user_project.deploy_keys.find_by(key: params[:key]) if key present key, with: Entities::SSHKey break end # Check for available deploy keys in other projects - key = current_user.accessible_deploy_keys.find_by(key: attrs[:key]) + key = current_user.accessible_deploy_keys.find_by(key: params[:key]) if key user_project.deploy_keys << key present key, with: Entities::SSHKey @@ -65,7 +64,7 @@ module API end # Create a new deploy key - key = DeployKey.new attrs + key = DeployKey.new(declared_params(include_missing: false)) if key.valid? && user_project.deploy_keys << key present key, with: Entities::SSHKey else |