Project

General

Profile

Actions

Enhancement #25478

open

Normalize authentication logs

Added by François ARMAND 3 months ago. Updated about 2 months ago.

Status:
Pending release
Priority:
N/A
Category:
Architecture - evolution
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
To do
Fix check:
Checked
Regression:
No

Description

We have a lot of differente loggers to log things about authentication, authorization, users, oidc, etc.
This is a mess that makes thing hard to change.

We need to normalize around only one logger: application.authentication

Loggers that should be migrated are at least:
- application.authorization
- application.user
- org.springframework.security.RudderProviderManager (because it's more linked to rudder authentication process than spring internals)

Perhaps there is others.

We should for now keep the auth-backends logger for the authentication backend plugin.
And user management should use the common logger.


Subtasks 2 (0 open2 closed)

User management - Enhancement #25484: Normalize authentication logs in pluginsReleasedFrançois ARMANDActions
Enhancement #25485: Normalize authentication logs in user-management 8.2 upmergeReleasedFrançois ARMANDActions

Related issues 1 (1 open0 closed)

Related to Rudder - Bug #25479: Users cleanup configuration is too strict on disabled usersPending releaseFrançois ARMANDActions
Actions #1

Updated by François ARMAND 3 months ago

  • Description updated (diff)
Actions #2

Updated by Clark ANDRIANASOLO 3 months ago

  • Related to Bug #25479: Users cleanup configuration is too strict on disabled users added
Actions #3

Updated by Clark ANDRIANASOLO 3 months ago

  • Subtask #25484 added
Actions #4

Updated by Clark ANDRIANASOLO 3 months ago

  • Status changed from New to In progress
Actions #5

Updated by Clark ANDRIANASOLO 3 months ago

  • Subtask #25485 added
Actions #6

Updated by Clark ANDRIANASOLO 3 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Clark ANDRIANASOLO to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/5883
Actions #7

Updated by Clark ANDRIANASOLO 3 months ago

  • Status changed from Pending technical review to Pending release
Actions #9

Updated by François ARMAND 3 months ago

  • Fix check changed from To do to Checked
2024-09-18 15:10:43+0000 DEBUG application.authentication - Rudder authentication attempt for principal 'rootadmin' with backend 'rootAdmin': success
Actions #10

Updated by Vincent MEMBRÉ 2 months ago

This bug has been fixed in Rudder 8.1.7 which was released today.

Actions

Also available in: Atom PDF