Skip to content

Update documentation OIDC#12153

Open
danduk82 wants to merge 1 commit intomasterfrom
update_doc_oauth
Open

Update documentation OIDC#12153
danduk82 wants to merge 1 commit intomasterfrom
update_doc_oauth

Conversation

@danduk82
Copy link
Copy Markdown
Member

No description provided.

@danduk82 danduk82 requested review from Copilot and sbrunner and removed request for Copilot July 15, 2025 11:56

This comment was marked as outdated.

@sbrunner sbrunner requested a review from Copilot July 15, 2025 13:22
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the OIDC integration guide by adding more context to the Client ID and refining the Scope instruction.

  • Clarify that the Client ID refers to the PKCE application's ID.
  • Simplify the Scope instruction wording.
Comments suppressed due to low confidence (2)

doc/integrator/authentication_oidc.rst:183

  • [nitpick] Consider simplifying this for clarity, e.g., 'Set Client ID to <client_id> (the PKCE application ID).' to avoid redundancy.
* Set ``Client ID`` to ``<client_id>``. This is the application ID of the PKCE application.

doc/integrator/authentication_oidc.rst:184

  • [nitpick] The phrase 'the following string' is unnecessary—consider simplifying to 'Set Scope to openid profile email.'
* Set ``Scope`` to the following string ``openid profile email``.

@sbrunner sbrunner changed the title Update documentation oidc Update documentation OIDC Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants