Project

General

Profile

Actions
Copy link

Bug #28452

closed

ADR for proper initialization of query context in Lift snippets

Bug #28452: ADR for proper initialization of query context in Lift snippets

Added by Clark ANDRIANASOLO 7 months ago. Updated 6 months ago.

Status:
Released
Priority:
N/A
Assignee:
Pauline IOGNA
Category:
Security
Target version:
8.3.10
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Very Small
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:
No

Description

There are a lot of usage for the QueryContext (tenants datatype) in Lift snippets.
But having a CurrentUser that exposes a dumb one is not the right way to obtain the user : it causes issues described in #26605.

We should avoid using CurrentUser as possible, and use dedicated way to access QueryContext in snippets (as in #28404)

Related issues 2 (0 open — 2 closed)

Related to Rudder - Bug #26605: CurrentUser.queryContext is null when used in a ZIO forReleasedClark ANDRIANASOLOActions
Related to Rudder - Bug #28404: CurrentUser can have unknown actor and query context in snippetsReleasedPauline IOGNAActions

Updated by Clark ANDRIANASOLO 7 months ago Actions
Copy link
 #1

  • Related to Bug #26605: CurrentUser.queryContext is null when used in a ZIO for added

Updated by Clark ANDRIANASOLO 7 months ago Actions
Copy link
 #2

  • Related to Bug #28404: CurrentUser can have unknown actor and query context in snippets added

Updated by Clark ANDRIANASOLO 7 months ago Actions
Copy link
 #3

  • Status changed from New to In progress

Updated by Clark ANDRIANASOLO 7 months ago Actions
Copy link
 #4

  • Subject changed from ADR for proper initialization of snippets query context to ADR for proper initialization of query context in Lift snippets
  • Status changed from In progress to New

Updated by Clark ANDRIANASOLO 7 months ago Actions
Copy link
 #5

  • Description updated (diff)

Updated by Clark ANDRIANASOLO 7 months ago Actions
Copy link
 #6

  • Status changed from New to Pending technical review
  • Assignee changed from Clark ANDRIANASOLO to Pauline IOGNA
  • Pull Request set to https://github.com/Normation/rudder/pull/6959

Updated by Clark ANDRIANASOLO 7 months ago Actions
Copy link
 #7

  • Status changed from Pending technical review to Pending release

Updated by François ARMAND 7 months ago Actions
Copy link
 #8

  • Fix check changed from To do to Checked

Updated by Vincent MEMBRÉ 6 months ago Actions
Copy link
 #9

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 8.3.10, 9.0.5 and 9.1.0~rc1 which were released today.

Actions
Copy link

Also available in: PDF Atom