Skip to content

PG-2238 - Add the OIDC topic#930

Merged
Andriciuc merged 9 commits into
18from
add-oidc-validator-topic
Jun 12, 2026
Merged

PG-2238 - Add the OIDC topic#930
Andriciuc merged 9 commits into
18from
add-oidc-validator-topic

Conversation

@Andriciuc

Copy link
Copy Markdown
Contributor

This PR reorganizes the authentication methods in Solutions, adding a folder to future proof the authentication methods, and adds the OIDC topic.

This PR reorganizes the authentication methods in Solutions, adding a folder to future proof the authentication methods, and adds the OIDC topic.
- add content to oidc, which includes beginner explanations for authentication flow, why use OIDC and standard setup from the PG side.
- Add buttons to overview, and add a lot of links to the oidc chapter for all of our sources.
Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/overview.md Outdated
- Proper installation steps, updated step 1 with readme info and remove step 2
- Reordered TOC to put OIDC first
- create a snippet folder with proper oidc authentication flow which can be modified any time to render immediately as opposed to having an svg, using mermaid.
- standardize the external identity providers, making a list of them is too bloaty.

@dutow dutow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is the oidc-auth-flow.svg still needed?

Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/oidc.md Outdated
Comment thread docs/solutions/authentication/oidc.md Outdated
- Removed important note at the end as it's redundant
- Cleaned step 1 to make it clearer how users can install the package which is already bundled
- Rewrote examples in step 3 to be clearer
@dutow

dutow commented Jun 2, 2026

Copy link
Copy Markdown

Is the oidc-auth-flow.svg still needed?

This question is still open, that seems to be a leftover file to me, we don't need it now with mermaid.

@Andriciuc

Copy link
Copy Markdown
Contributor Author

Is the oidc-auth-flow.svg still needed?

This question is still open, that seems to be a leftover file to me, we don't need it now with mermaid.

I believe we can keep it, it's legacy content and it's good to have it for historical reference even though we don't use it anymore.

@dutow

dutow commented Jun 8, 2026

Copy link
Copy Markdown

I believe we can keep it, it's legacy content and it's good to have it for historical reference even though we don't use it anymore.

It is a newly added file in this PR? I don't see how that's historical context (or why we need historical context outside the git history)

@Andriciuc

Copy link
Copy Markdown
Contributor Author

I believe we can keep it, it's legacy content and it's good to have it for historical reference even though we don't use it anymore.

It is a newly added file in this PR? I don't see how that's historical context (or why we need historical context outside the git history)

Yep basically git history. But I'll cut it out now.

@Andriciuc Andriciuc temporarily deployed to add-oidc-validator-topic - postgresql-docs-preview-18 PR #930 June 8, 2026 10:07 — with Render Destroyed
@Andriciuc Andriciuc merged commit a0e13a6 into 18 Jun 12, 2026
1 check passed
@Andriciuc Andriciuc deleted the add-oidc-validator-topic branch June 12, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants