[DNS] Update proxy status page and improve information overload #19728
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
RebeccaTamachiro
requested review from
a team,
GregBrimble,
ToriLindsay,
WalshyDev,
dcpena,
irvinebroque,
maxvp and
mikenomitch
as code owners
github-actions
bot
added
product:byoip
Deploying cloudflare-docs with
|
| Latest commit: |
c49ad95
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2e28c851.cloudflare-docs-7ou.pages.dev |
| Branch Preview URL: | https://rebecca-dns-proxy-status-rev.cloudflare-docs-7ou.pages.dev |
github-actions
bot
assigned
maxvp,
dcpena,
GregBrimble,
irvinebroque,
mikenomitch,
ToriLindsay and
WalshyDev
Contributor
Contributor
|
Preview URL: https://e5bafff6.preview.developers.cloudflare.com Files with changes (up to 15)
|
RebeccaTamachiro
requested review from
a team,
KianNH,
angelampcosta,
bjesus,
haleycode,
jason-cf,
jonnyparris,
ngayerie,
pedrosousa,
renandincer,
securitypedant,
simonabadoiu,
third774,
tsmith512 and
zeinjaber
as code owners
marciocloudflare
approved these changes
Feb 11, 2025
| <Details header="Example"> | ||
|
|
||
| <Example> | ||
| DNS management for **example.com**: |
Contributor
There was a problem hiding this comment.
This seems to be the same table from the partial file; you could use it here as well
Contributor
Author
There was a problem hiding this comment.
They're slightly different, bc this one is for records with same name (blog and blog) vs blog and shop in the partial
Contributor
There was a problem hiding this comment.
You could use variables and change the names accordingly :)
|
|
||
| In addition to potentially exposing your origin IP addresses to bad actors and [DDoS attacks](https://www.cloudflare.com/learning/ddos/what-is-a-ddos-attack/), leaving your records as **DNS-only** means that Cloudflare cannot [optimize, cache, and protect](/fundamentals/concepts/how-cloudflare-works/) requests to your application or provide analytics on those requests. | ||
|
|
||
| <Render file="mix-proxied-and-unproxied" product="dns" /> No newline at end of file |
Contributor
There was a problem hiding this comment.
In this partial, records are monospaced - in between backticks - but throughout the page you didn't monospaced any record types
Co-authored-by: marciocloudflare <[email protected]>
jonesphillip
pushed a commit
that referenced
this pull request
Feb 12, 2025
* Recap what are DNS records, remove image, and add Benefits h3 * Briefly explain the example pointing to Fundamentals for more * Remove repeated explanation and details on egress IPs * Repurpose proxied-records h2 and update anchor links * Add dedicated folder to move proxying higher on the sidenav * Add dedicated page to proxying limitations * Add Secondary DNS with pre-signed DNSSEC limitation * Explicitly call out DNS-only on pre-signed DNSSEC warning * Replace link to Secondary DNS in limitations.mdx * Small text tweaks for consistency * Call out example where CNAME shouldn't be proxied * Move proxied-dns-records to index.mdx for name consistency * Adjust pcx_content_type accordingly * Improve linkning to Fundamentals from limitations.mdx * Apply PM feedback and improve passive voice * Edit explanation of predefined TTL as per PM comment * Fix links with specific anchors to limitations * Find and replace remaining links that kept valid anchors * Further adjust specific links to #proxied-records * Add redirect from previous buried page to new one * Adding some changes left unstaged * Fix broken links * Apply suggestions from code review Co-authored-by: marciocloudflare <[email protected]> --------- Co-authored-by: marciocloudflare <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
PCX-14292: Make proxied-dns-records more discoverable and beginner friendly
PCX-11582: Add note about proxy status for presigned zones
Documentation checklist