Skip to content

disclaimer fix to the release branch - #3232

Merged
SantanM merged 3 commits into
release/v0.18.0-betafrom
SantanM/internal-64_disclaimer-fix
Aug 28, 2026
Merged

disclaimer fix to the release branch#3232
SantanM merged 3 commits into
release/v0.18.0-betafrom
SantanM/internal-64_disclaimer-fix

Conversation

@SantanM

@SantanM SantanM commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Original PR intended for the develop branch is here

Merge Checklist

Please cross check this list if additions / modifications needs to be done on top of your core changes and tick them off. Reviewer can as well glance through and help the developer if something is missed out.

  • Automated Tests (Jasmine integration tests, Unit tests, and/or Performance tests)
  • Updated Manual tests / Demo Config
  • Documentation (Application guide, Admin guide, Markdown, Readme and/or Wiki)
  • Verified that local development environment is working with latest changes (integrated with latest develop branch)
  • following best practices in code review doc

ohdsi-trex and others added 3 commits August 28, 2026 13:16
@ohdsi-trex @SantanM
fix: disclaimer audit logging and logout flow
48f4126 
(cherry picked from commit b1413f5)
@ohdsi-trex @SantanM
chore: remove disclaimer audit plans from PR
95bf941 
(cherry picked from commit 26eaab6)
@SantanM
Log disclaimer audit in AUDITLOG format with IDP identity fallback
c0a6871 
Resolve the user from thirdPartyToken.oid, falling back to
GATEWAY__IDP_SUBJECT_PROP, the oid claim, then the Logto subject.

(cherry picked from commit 7fe9dc9)
@SantanM SantanM changed the title Santan m/internal 64 disclaimer fix disclaimer fix to the release branch Aug 28, 2026
@SantanM
SantanM merged commit defee57 into release/v0.18.0-beta Aug 28, 2026
112 of 121 checks passed
@SantanM
SantanM deleted the SantanM/internal-64_disclaimer-fix branch August 28, 2026 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@p-hoffmann p-hoffmann p-hoffmann approved these changes
@hengxian-jiang hengxian-jiang Awaiting requested review from hengxian-jiang hengxian-jiang is a code owner
@suwarnoong suwarnoong Awaiting requested review from suwarnoong suwarnoong is a code owner
@brandantck brandantck Awaiting requested review from brandantck brandantck is a code owner
@csafreen csafreen Awaiting requested review from csafreen csafreen is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@SantanM @p-hoffmann @ohdsi-trex