FHIDev / fhi.helseid

Fhi.HelseId component for accessing NHN HelseId
MIT License
7 stars 5 forks source link

Adjust scope logging, do not validate security level claims if user is not authenticated #392

Closed martinmine closed 4 months ago

martinmine commented 4 months ago

ApiMultiScopeHandler: This class contained verbose logging that caused log files to become unreasonably large and hard to make sense of. This makes more sense to be enabled in a debugging context, so log level is set to Trace instead of Information. If we get an unauthorized response we will nevertheless log the claims as an error log event.

SecurityLevelClaimHandler: Do not validate if the user is not authenticated and log that no user is authenticated. Previously, if we sent a request with an empty authorization value or missing cookie, it would result in an error log event that no security level claim was found.

sonarcloud[bot] commented 4 months ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud