Skip to content

docs(openid-connect): re-port with Admin API, ADC, and Ingress Controller tabs#13247

Merged
Yilialinn merged 12 commits into
apache:masterfrom
Yilialinn:docs/update-openid-connect
Apr 30, 2026
Merged

docs(openid-connect): re-port with Admin API, ADC, and Ingress Controller tabs#13247
Yilialinn merged 12 commits into
apache:masterfrom
Yilialinn:docs/update-openid-connect

Conversation

@Yilialinn

Copy link
Copy Markdown
Contributor

Re-ports the openid-connect plugin documentation with Admin API, ADC, and Ingress Controller tab structure, updated attribute table, canonical link, and breaking change note for ssl_verify default value change in APISIX 3.16.0.

@dosubot dosubot Bot added size:XL This PR changes 500-999 lines, ignoring generated files. doc Documentation things labels Apr 17, 2026
@Yilialinn
Yilialinn requested a review from Copilot April 17, 2026 09:19

Copilot AI 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.

Pull request overview

This PR re-ports the openid-connect plugin documentation to the newer documentation structure (Admin API / ADC / Ingress Controller tabs), refreshes the attribute tables, and documents the ssl_verify default change as a breaking change (APISIX 3.16.0).

Changes:

  • Adds Docusaurus tabbed examples for Admin API, ADC, and Ingress Controller configurations.
  • Reworks and expands the attribute tables, including updated encrypt_fields notes and environment variable examples.
  • Updates/expands troubleshooting content and adds a “User Info” section (EN/ZH).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 8 comments.

File Description
docs/zh/latest/plugins/openid-connect.md Reworked ZH plugin doc structure, attributes table, and examples with tabs.
docs/en/latest/plugins/openid-connect.md Reworked EN plugin doc structure, attributes table, and examples with tabs; adds “User Info” section.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/zh/latest/plugins/openid-connect.md Outdated
Comment thread docs/zh/latest/plugins/openid-connect.md Outdated
Comment thread docs/zh/latest/plugins/openid-connect.md Outdated
Comment thread docs/zh/latest/plugins/openid-connect.md
Comment thread docs/en/latest/plugins/openid-connect.md Outdated
Comment thread docs/en/latest/plugins/openid-connect.md Outdated
Comment thread docs/en/latest/plugins/openid-connect.md
Comment thread docs/en/latest/plugins/openid-connect.md Outdated
@Yilialinn
Yilialinn requested a review from kayx23 April 21, 2026 08:58

@kayx23 kayx23 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The new Kubernetes examples still reference an httpbin backend, but unlike the other plugin docs in this batch the snippet does not define that Service/Upstream in the example itself. As written, both the Gateway API backendRefs: - name: httpbin example and the APISIX Ingress serviceName: httpbin example are incomplete and will not work unless readers already created that backend separately.

@Yilialinn
Yilialinn requested a review from kayx23 April 22, 2026 06:27
Comment thread docs/en/latest/plugins/openid-connect.md
@Yilialinn
Yilialinn merged commit a3c4e8c into apache:master Apr 30, 2026
11 of 12 checks passed
@Yilialinn
Yilialinn deleted the docs/update-openid-connect branch April 30, 2026 03:23
francescodedomenico added a commit to francescodedomenico/apisix that referenced this pull request May 15, 2026
The previous merge of master into feat/oidc-session-cookie accepted
ours for the EN openid-connect.md attribute table, dropping the
re-port improvements from apache#13247 (a3c4e8c) while keeping only the
branch session.cookie.* additions. Restore master's table and
re-inject the session.cookie.{name,path,lifetime,<other>} rows and
the updated session.cookie description.

The ZH docs were merged correctly and are unchanged.
wistefan pushed a commit to wistefan/apisix that referenced this pull request Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Documentation things size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants