Skip to content

Bugfix: id_token claims missing when sub is not consistent#931

Merged
esune merged 5 commits intomainfrom
hotfix/v2.3.4-rebased
Dec 5, 2025
Merged

Bugfix: id_token claims missing when sub is not consistent#931
esune merged 5 commits intomainfrom
hotfix/v2.3.4-rebased

Conversation

@esune
Copy link
Member

@esune esune commented Dec 4, 2025

Same PR as #930, but to main.

Fixes an edge case where claims would be missing from the id_token when the proof-request did not specify which attribute to use as sub and the setting to generate a consistent identifier was set to false.

esune added 4 commits December 5, 2025 00:22
…age (#924)

* Fix userinfo flow for in-memory storage

Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>

* Use in-memory storage for single pod mode

Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>

* Bump project version

Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>

* Update tests

Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>

---------

Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>
Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>
Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>
Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>
@esune esune requested review from Gavinok and jamshale December 4, 2025 23:28
Signed-off-by: Emiliano Suñé <emiliano.sune@gmail.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 19947427668

Details

  • 6 of 6 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 87.727%

Totals Coverage Status
Change from base Build 19931823248: 0.01%
Covered Lines: 1930
Relevant Lines: 2200

💛 - Coveralls

@esune esune merged commit 01a9c51 into main Dec 5, 2025
6 checks passed
@esune esune deleted the hotfix/v2.3.4-rebased branch December 5, 2025 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants