Skip to content
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

IdentityServer Error LogLevel #1547

Open
ruudhe opened this issue Apr 30, 2024 · 1 comment
Open

IdentityServer Error LogLevel #1547

ruudhe opened this issue Apr 30, 2024 · 1 comment
Assignees

Comments

@ruudhe
Copy link

ruudhe commented Apr 30, 2024

Which version of Duende IdentityServer are you using?
IdentityServer 7.0.3
BFF 2.2.0

Which version of .NET are you using?
net8.0

Describe the bug
On a high volume public facing deployment our logs are spammed with IdentityServer Errors which are generated by security scans or wrong user input. The validators used in identityserver are logging all invalid requests as an Error, for example:

AuthorizeRequestValidator

LogError("client_id is missing or too long", request);

LogError("redirect_uri is missing or too long", request);

LogError("Invalid redirect_uri", redirectUri, request);

ProtectedDataMessageStore

Logger.LogError(ex, "Exception reading protected message");

Expected behavior
We would like to request a change that the logging level for these kind of log statements can be configured from Error to Warning so error reporting would not give so much errors due to improper usage / url scanning.

@AndersAbel
Copy link
Member

Thanks for your feedback, I think this makes sense. I'm transferring the issue to the main IdentityServer repo for triage/planning.

@AndersAbel AndersAbel transferred this issue from DuendeSoftware/Support May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants