Identityserver4: Decrease log level for failed user login

Created on 13 Nov 2017  路  5Comments  路  Source: IdentityServer/IdentityServer4

Issue

There was an issue in Identity Server 3 about log level in TokenRequestValidator. I have the same problem at Identity Server 4, and it would be nice to decrease log level about failed user login to Warning or lower. Or maybe someone can suggest how to do it from my side?

core feature request
Source
picture RakhmatovR
👍2

Most helpful comment

It would be neat to dial down some of the messages in the TokenValidator as well; I'd not have things like 'token expired' as errors.

Thanks!

picture ssarj on 29 Nov 2017
👍3

All 5 comments

Which exact message are we looking for?

leastprivilege picture leastprivilege on 13 Nov 2017

@leastprivilege
https://github.com/IdentityServer/IdentityServer4/blob/dev/src/IdentityServer4/Validation/TokenRequestValidator.cs

420 line:

LogError("User authentication failed: {error}", errorDescription ?? resourceOwnerContext.Result.Error);
RakhmatovR picture RakhmatovR on 13 Nov 2017

https://github.com/IdentityServer/IdentityServer4/blob/3d738e96263d3b2728bdde9474715d9c392b2875/src/IdentityServer4/Validation/TokenRequestValidator.cs#L413-L420

OK - we'll discuss this. @brockallen

leastprivilege picture leastprivilege on 13 Nov 2017

It would be neat to dial down some of the messages in the TokenValidator as well; I'd not have things like 'token expired' as errors.

Thanks!

ssarj picture ssarj on 29 Nov 2017
👍3

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock[bot] picture lock[bot] on 13 Jan 2020
Was this page helpful?
0 / 5 - 0 ratings

Related issues

eshorgan picture eshorgan  路  3Comments
wangkanai picture wangkanai  路  3Comments
krgm03 picture krgm03  路  3Comments
mackie1001 picture mackie1001  路  3Comments
not-good-with-usernames picture not-good-with-usernames  路  3Comments