Merge branch 'main' into io-to-file

Gemfile

@@ -123,7 +123,7 @@ gem 'delayed_job_heartbeat_plugin'
 gem 'representable'
 
 # Keyword search
-gem 'keyword_search', '~> 1.5.0'
+gem 'keyword_search'
 
 # ToS/PP management
 gem 'fine_print', github: 'lml/fine_print', ref: '636023f68e95196dffaf295bfad3ad8051c23542'

Gemfile.lock

@@ -863,7 +863,7 @@ DEPENDENCIES
   jobba
   jquery-rails
   json-jwt
-  keyword_search (~> 1.5.0)
+  keyword_search
   knockoutjs-rails
   launchy
   lev!

app/controllers/api/v1/users_controller.rb

@@ -264,7 +264,7 @@ def get_sso_token(application, user)
     access_token = Doorkeeper::AccessToken.find_or_create_for(
       application: application,
       resource_owner: user.id,
-      scopes: '',
+      scopes: Doorkeeper.config.default_scopes,
       expires_in: SSO_TOKEN_INITIAL_DURATION,
       use_refresh_token: false,
     )
@@ -278,7 +278,7 @@ def get_sso_token(application, user)
     access_token = Doorkeeper::AccessToken.create_for(
       application: application.id,
       resource_owner: user.id,
-      scopes: '',
+      scopes: Doorkeeper.config.default_scopes,
       expires_in: SSO_TOKEN_INITIAL_DURATION,
       use_refresh_token: false
     )

app/routines/search_users.rb

@@ -47,7 +47,7 @@ def exec(query, options={})
       ]
     )
 
-    KeywordSearch.search(query) do |with|
+    KeywordSearch.search(query || '') do |with|
 
       with.default_keyword :any
 

spec/controllers/api/v1/users_controller_spec.rb

@@ -58,6 +58,12 @@
   let(:is_not_gdpr_location) { nil }
 
   context "index" do
+    it "does not explode when called without params" do
+      api_get :index, trusted_application_token
+      expect(response.code).to eq('200')
+      expect(response.body_as_hash).to match({ total_count: User.count, items: [] })
+    end
+
     it "returns a single result well" do
       api_get :index, trusted_application_token, params: { q: 'first_name:bob last_name:Michaels' }
       expect(response.code).to eq('200')