-
Notifications
You must be signed in to change notification settings - Fork 6k
Provide extendend SAML logging (like SAMLDefaultLogger) #11592
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Hi, @habizzle. Can you tell me more about what you are trying to do? I ask since it's not clear to me how introducing a logging component will change the log detail level.
Should the authentication provider log more advanced information and if so, where? If you and I can come to an agreement on that, the best solution could be a PR so that everyone gets the benefit of additional detail. |
If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed. |
The old library allowed to log full SAML requests and responses to server logs. It was useful for investigating issues with SAML login reported by customers (in my case we integrate with our customer's Identity Providers). |
Sorry for the delay, @mjanczykowski. I think adding requests and responses to trace logging sounds reasonable. |
Expected Behavior
We would like to have the possibility to easily log SAML responses as with the (previous)
SAMLLogger
-API.Current Behavior
We did not find a "standard" way of logging details of the SAML response.
Context
We are currently migrating from Spring Security SAML Extension to the new Spring Security Service Provider library. We have used a
SAMLDefaultLogger
bean to have an extensive logging of the SAML response (e.g. in the oldSAMLAuthenticationProvider
).In the new library we did not find something like the old
SAMLLogger
-API and in the newOpenSamlAuthenticationProvider
, the logger does only log pretty basic information.Are you planning to port this feature or do you recommend overriding e.g. the
AuthenticationProvider
for adding more logging?Thanks!
The text was updated successfully, but these errors were encountered: