Prevent status webservices from being returned on the providers endpoint

Author: mFelgate

CHANGELOG.md

@@ -14,6 +14,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Changed
 - Reduces debug log verbosity.
   [cyberark/conjur#2639](https://github.com/cyberark/conjur/pull/2639)
+- Remove status webservices from providers endpoints
+  [cyberark/conjur#2640](https://github.com/cyberark/conjur/pull/2640)
 
 ## [1.18.1] - 2022-08-01
 

app/db/repository/authenticator_repository.rb

@@ -8,11 +8,18 @@ def initialize(data_object:, resource_repository: ::Resource, logger: Rails.logg
       end
 
       def find_all(type:, account:)
+        identifier = Sequel.function(:identifier, :resource_id)
+
         @resource_repository.where(
           Sequel.like(
             :resource_id,
             "#{account}:webservice:conjur/#{type}/%"
           )
+        ).where(
+          Sequel.like(
+            identifier,
+            %r{conjur/(authn(?:-[^/]+)?(?:/[^/]+)?)$}
+          )
         ).all.map do |webservice|
           load_authenticator(account: account, id: webservice.id.split(':').last, type: type)
         end.compact

spec/app/db/repository/authenticator_repository_spec.rb

@@ -89,6 +89,21 @@
               ::Role['rspec:policy:conjur/authn-oidc/baz-abc123'].destroy
             end
           end
+
+          context 'when webservices status are presents' do
+            before(:each) do
+                ::Resource.create(
+                  resource_id: "rspec:webservice:conjur/authn-oidc/foo-abc123/status",
+                  owner_id: "rspec:policy:conjur/authn-oidc/foo-abc123"
+                )
+              end
+
+            it { expect(repo.find_all(type: 'authn-oidc', account: 'rspec').length).to eq(2) }
+
+            after(:each) do
+              ::Resource['rspec:webservice:conjur/authn-oidc/foo-abc123/status'].destroy
+            end
+          end
         end
 
         after(:each) do