diff --git a/library/EngineBlock/Corto/Module/Service/AssertionConsumer.php b/library/EngineBlock/Corto/Module/Service/AssertionConsumer.php index cc7db2f11..0ca31bce6 100644 --- a/library/EngineBlock/Corto/Module/Service/AssertionConsumer.php +++ b/library/EngineBlock/Corto/Module/Service/AssertionConsumer.php @@ -193,7 +193,7 @@ public function serve($serviceName, Request $httpRequest) return; } - $log->info('Handle Stepup authentication callout', array('key_id' => $receivedRequest->getId())); + $log->info('Handle Stepup authentication callout'); // Add Stepup authentication step $currentProcessStep = $this->_processingStateHelper->addStep( diff --git a/library/EngineBlock/Corto/Module/Service/StepupAssertionConsumer.php b/library/EngineBlock/Corto/Module/Service/StepupAssertionConsumer.php index 32d640c80..62b7d55d6 100644 --- a/library/EngineBlock/Corto/Module/Service/StepupAssertionConsumer.php +++ b/library/EngineBlock/Corto/Module/Service/StepupAssertionConsumer.php @@ -93,7 +93,7 @@ public function serve($serviceName, Request $httpRequest) $receivedLoa = $this->_stepupGatewayCallOutHelper->getEbLoa($receivedResponse->getAssertion()->getAuthnContextClassRef()); $this->updateProcessingStateLoa($receivedRequest, $receivedResponse, $receivedLoa); - $log->warning('After Stepup authentication update received LoA', array('key_id' => $receivedRequest->getId(), 'result' => $receivedLoa)); + $log->warning('After Stepup authentication update received LoA', ['result' => $receivedLoa]); } catch (EngineBlock_Corto_Exception_ReceivedErrorStatusCode $e) { // The user is allowed to continue upon subcode: NoAuthnContext when the SP is configured with the coin: coin:stepup:allow_no_token == true @@ -108,7 +108,7 @@ public function serve($serviceName, Request $httpRequest) // Update the AuthnContextClassRef to LoA 1 $mappedLoa = $this->_stepupGatewayCallOutHelper->getStepupLoa1(); $this->updateProcessingStateLoa($receivedRequest, $receivedResponse, $mappedLoa); - $log->warning('After failed Stepup authentication set LoA to Loa1', array('key_id' => $receivedRequest->getId(), 'result' => $mappedLoa)); + $log->warning('After failed Stepup authentication set LoA to Loa1', ['result' => $mappedLoa]); } if ($checkResponseSignature) { @@ -125,7 +125,7 @@ public function serve($serviceName, Request $httpRequest) $this->_server->getRepository() ); - $log->info('Handled Stepup authentication callout successfully', array('key_id' => $receivedRequest->getId())); + $log->info('Handled Stepup authentication callout successfully'); // Get active request $processStep = $this->_processingStateHelper->getStepByRequestId($receivedRequest->getId(), ProcessingStateHelperInterface::STEP_STEPUP); @@ -193,7 +193,7 @@ private function handleInvalidGatewayResponse(EngineBlock_Corto_Exception_Receiv switch (true) { case ($e->getFeedbackStatusCode() === 'Responder/AuthnFailed' && $e->getFeedbackStatusMessage() === 'Authentication cancelled by user'): // user cancelled - $log->info('User cancelled Stepup authentication callout', array('key_id' => $receivedRequest->getId(), 'result' => $e->getFeedbackInfo())); + $log->info('User cancelled Stepup authentication callout', ['result' => $e->getFeedbackInfo()]); throw new EngineBlock_Corto_Exception_UserCancelledStepupCallout( 'User cancelled Stepup authentication callout', @@ -204,7 +204,7 @@ private function handleInvalidGatewayResponse(EngineBlock_Corto_Exception_Receiv // invalid loa level // should continue if no valid token is allowed - $log->warning('Unmet loa level for Stepup authentication callout, trying allow no token', array('key_id' => $receivedRequest->getId(), 'result' => $e->getFeedbackInfo())); + $log->warning('Unmet loa level for Stepup authentication callout, trying allow no token', ['result' => $e->getFeedbackInfo()]); // check if no token allowed $processStep = $this->_processingStateHelper->getStepByRequestId($receivedRequest->getId(), ProcessingStateHelperInterface::STEP_STEPUP); @@ -216,7 +216,7 @@ private function handleInvalidGatewayResponse(EngineBlock_Corto_Exception_Receiv if ($this->_stepupGatewayCallOutHelper->allowNoToken($idp, $sp)) { - $log->warning('Allow no token allowed from sp/idp configuration, continuing', array('key_id' => $receivedRequest->getId(), 'result' => $e->getFeedbackInfo())); + $log->warning('Allow no token allowed from sp/idp configuration, continuing', ['result' => $e->getFeedbackInfo()]); return; } @@ -226,7 +226,7 @@ private function handleInvalidGatewayResponse(EngineBlock_Corto_Exception_Receiv ); } - $log->warning('Invalid status returned from Stepup authentication gateway', array('key_id' => $receivedRequest->getId(), 'result' => $e->getFeedbackInfo())); + $log->warning('Invalid status returned from Stepup authentication gateway', ['result' => $e->getFeedbackInfo()]); throw new EngineBlock_Corto_Exception_InvalidStepupCalloutResponse( sprintf(