Convert some trace logs to warnings in OAuth flows (#16213)

* Less trace, more warn in OpenID provider

* Less trace, more warn in OAuth provider
This commit is contained in:
Aiden Foxx
2022-10-29 21:59:50 +02:00
committed by GitHub
parent 1de75acdc0
commit 2adbb3bfd2
2 changed files with 4 additions and 4 deletions

View File

@@ -104,7 +104,7 @@ export class OAuth2AuthDriver extends LocalAuthDriver {
async getUserID(payload: Record<string, any>): Promise<string> {
if (!payload.code || !payload.codeVerifier) {
logger.trace('[OAuth2] No code or codeVerifier in payload');
logger.warn('[OAuth2] No code or codeVerifier in payload');
throw new InvalidCredentialsException();
}
@@ -150,7 +150,7 @@ export class OAuth2AuthDriver extends LocalAuthDriver {
// Is public registration allowed?
if (!allowPublicRegistration) {
logger.trace(`[OAuth2] User doesn't exist, and public registration not allowed for provider "${provider}"`);
logger.warn(`[OAuth2] User doesn't exist, and public registration not allowed for provider "${provider}"`);
throw new InvalidCredentialsException();
}

View File

@@ -117,7 +117,7 @@ export class OpenIDAuthDriver extends LocalAuthDriver {
async getUserID(payload: Record<string, any>): Promise<string> {
if (!payload.code || !payload.codeVerifier) {
logger.trace('[OpenID] No code or codeVerifier in payload');
logger.warn('[OpenID] No code or codeVerifier in payload');
throw new InvalidCredentialsException();
}
@@ -173,7 +173,7 @@ export class OpenIDAuthDriver extends LocalAuthDriver {
// Is public registration allowed?
if (!allowPublicRegistration || !isEmailVerified) {
logger.trace(`[OpenID] User doesn't exist, and public registration not allowed for provider "${provider}"`);
logger.warn(`[OpenID] User doesn't exist, and public registration not allowed for provider "${provider}"`);
throw new InvalidCredentialsException();
}