Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Errors] Update 4xx-client-error.mdx #18887

Merged
merged 4 commits into from
Jan 2, 2025
Merged

[Errors] Update 4xx-client-error.mdx #18887

merged 4 commits into from
Jan 2, 2025

Conversation

ngayerie
Copy link
Contributor

@ngayerie ngayerie commented Dec 20, 2024

Summary

Updating the 403 documentation to mention SNI mismatch and validation checks, PCX-15123

Copy link

cloudflare-workers-and-pages bot commented Dec 20, 2024

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 66364bf
Status: ✅  Deploy successful!
Preview URL: https://00b57d9e.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://ngayerie-patch-11.cloudflare-docs-7ou.pages.dev

View logs

Copy link
Contributor

@dom-cf dom-cf left a comment

Choose a reason for hiding this comment

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

Suggesting moving Validation to the standard 403 list as it returns a branded 1012 Access Denied.
The SNI/Host mismatch is the only scenario that returns an unstyled 403 as far as I am aware.

@ngayerie ngayerie merged commit 257b2ed into production Jan 2, 2025
12 checks passed
@ngayerie ngayerie deleted the ngayerie-patch-11 branch January 2, 2025 13:20
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.

6 participants