Skip to content

Conversation

AxelNennker
Copy link
Collaborator

What type of PR is this?

  • documentation

What this PR does / why we need it:

Clarify that the value of the "operatortoken" is a token created by the Operator, usually by the operator's Entitlement Server.
This value may an TS.43 operator token as defined in TS.43 v12 section 14 "Device App Authentication" or a Temporary Token also defined in GSMS TS.43.

Which issue(s) this PR fixes:

Fixes #296

Changelog input

 release-note

Additional documentation

docs

Copy link
Collaborator

@jpengar jpengar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change must be consistent throughout all ICM documents. CAMARA-API-access-and-user-consent.md mentions operator tokens in multiple places. This document also needs to be reviewed. However, CAMARA-ICM-examples.md looks fine as it is, as it neither mentions Operator Tokens nor contains examples of them.

@shilpa-padgaonkar
Copy link
Collaborator

@jpengar Thanks, we will add changes to the other docs as well as a part of this PR.

For the login_hint format-part we decided to leave it as-is for now.

WDYT?

@AxelNennker
Copy link
Collaborator Author

This change must be consistent throughout all ICM documents. CAMARA-API-access-and-user-consent.md mentions operator tokens in multiple places. This document also needs to be reviewed. However, CAMARA-ICM-examples.md looks fine as it is, as it neither mentions Operator Tokens nor contains examples of them.

I reviewed CAMARA-API-access-and-user-consent.md and all places where "operatortoken" state that the value is a "TS.43 token".

So, I think, CAMARA-API-access-and-user-consent.md does not need to be part of this PR.

Copy link
Collaborator

@jpengar jpengar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. We could include this change in the ICM public release of the Fall25 meta-release.

Copy link
Collaborator

@sebdewet sebdewet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jpengar jpengar added this to the Meta-release Fall25 milestone Jul 2, 2025
@jpengar jpengar merged commit 373ca18 into main Jul 4, 2025
1 check passed
@jpengar jpengar deleted the TS43-token branch August 18, 2025 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify and Generalize login_hint Usage to Include Operator and Temporary Tokens (TS.43) in ICM documentation
4 participants