Skip to content

[Security] Support new sap_id_typeclaim#1124

Merged
CharlesDuboisSAP merged 2 commits intomainfrom
sap_id_type
Mar 18, 2026
Merged

[Security] Support new sap_id_typeclaim#1124
CharlesDuboisSAP merged 2 commits intomainfrom
sap_id_type

Conversation

@CharlesDuboisSAP
Copy link
Contributor

@CharlesDuboisSAP CharlesDuboisSAP commented Mar 17, 2026

Context

New for SCI is now a sap_id_type claim. This claim is always in the JWT (id and access token).
With value app you can rely on a client authentication only and the sub is then always the client_id.

With value user you can rely on a User authentication and then you can rely that sub is the user identifier. This Identifier is called Subject Name Identifier in SCI and in the Admin UI customers can define this. Typically the userId, or email or user_uuid.

Feature scope:

  • Support new sap_id_type claim

Definition of Done

  • Functionality scope stated & covered
  • Tests cover the scope above
  • Error handling created / updated & covered by the tests above
  • Documentation updated
  • Release notes updated

@CharlesDuboisSAP CharlesDuboisSAP self-assigned this Mar 17, 2026
@CharlesDuboisSAP CharlesDuboisSAP added please merge Request to merge a pull request please review Request to review a pull request labels Mar 17, 2026
@CharlesDuboisSAP CharlesDuboisSAP changed the title sap_id_type [Security] Support new sap_id_typeclaim Mar 17, 2026
@strehle
Copy link
Member

strehle commented Mar 17, 2026

@torsten-sap FYI

@CharlesDuboisSAP CharlesDuboisSAP enabled auto-merge (squash) March 18, 2026 14:37
@CharlesDuboisSAP CharlesDuboisSAP merged commit 8aa4216 into main Mar 18, 2026
13 checks passed
@CharlesDuboisSAP CharlesDuboisSAP deleted the sap_id_type branch March 18, 2026 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

please merge Request to merge a pull request please review Request to review a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants