Merge pull request #2598 from cyberark/Fix-cucumber-bug

Fix cucumber bug
cyberark · Jul 11, 2022 · f18eec8 · f18eec8
2 parents 81f4c39 + 8d5c47f
commit f18eec8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/app/domain/authentication/handler/authentication_handler.rb b/app/domain/authentication/handler/authentication_handler.rb
@@ -71,18 +71,13 @@ def call(parameters:, request_ip:)
           account: parameters[:account],
           username: role.role_id.split(':').last
         )
-      rescue e
+      rescue => e
         log_audit_failure(parameters[:account], parameters[:service_id], request_ip, @authenticator_type, e)
         handle_error(e)
       end
 
       def handle_error(err)
-        # authentication_error = LogMessages::Authentication::AuthenticationError.new(err.inspect)
         @logger.warn(@authentication_error.new(err))
-
-        puts authentication_error
-        puts authentication_error
-        puts Errors::Authentication::Security::RoleNotFound
         case err
         when Errors::Authentication::Security::RoleNotAuthorizedOnResource
           puts "Errors::Authentication::Security::RoleNotAuthorizedOnResource"

diff --git a/dev/docker-compose.yml b/dev/docker-compose.yml
@@ -131,7 +131,7 @@ services:
       - [email protected]
       - KEYCLOAK_SECOND_APP_USER=bob.somebody
       - KEYCLOAK_SECOND_APP_USER_PASSWORD=bob
-      - KEYCLOAK_SECOND_APP_USER_EMAIL=bob[email protected]
+      - KEYCLOAK_SECOND_APP_USER_EMAIL=bob@conjur.net
       - KEYCLOAK_NON_CONJUR_APP_USER=not_in_conjur
       - KEYCLOAK_NON_CONJUR_APP_USER_PASSWORD=not_in_conjur
       - KEYCLOAK_NON_CONJUR_APP_USER_EMAIL=not_in_conjur