about summary refs log tree commit diff
path: root/app/controllers/api/v1
diff options
context:
space:
mode:
authorJennifer Glauche <=^.^=@github19.jglauche.de>2019-11-16 19:02:09 +0100
committerEugen Rochko <eugen@zeonfederated.com>2019-11-16 19:02:09 +0100
commitfd93a9c871e0c758603b5a939dc192aeefddab1d (patch)
tree5a43a1a5312a254bb28eef1b80788a23fd7b6083 /app/controllers/api/v1
parent286bf45d4cae7e6ea2c9164d861f020df6fa2a43 (diff)
make it not return http 400 when passing and empty source argument (#12259)
* make it not return http 400 when passing and empty source argument

* create a spec for the empty source hash bug

* compact checks for nil, empty? parameters

* use nil.blank? instead checking for nil
Diffstat (limited to 'app/controllers/api/v1')
-rw-r--r--app/controllers/api/v1/accounts/credentials_controller.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/app/controllers/api/v1/accounts/credentials_controller.rb b/app/controllers/api/v1/accounts/credentials_controller.rb
index e77f57910..64b5cb747 100644
--- a/app/controllers/api/v1/accounts/credentials_controller.rb
+++ b/app/controllers/api/v1/accounts/credentials_controller.rb
@@ -25,7 +25,7 @@ class Api::V1::Accounts::CredentialsController < Api::BaseController
   end
 
   def user_settings_params
-    return nil unless params.key?(:source)
+    return nil if params[:source].blank?
 
     source_params = params.require(:source)