Skip to content

Conversation

@PaulAngus
Copy link
Member

improve information to external DNS note and reformat line widths.

@PaulAngus
Copy link
Member Author

request docbuild

@apache apache deleted a comment from ACSGitBot May 28, 2020
@apache apache deleted a comment from ACSGitBot May 28, 2020
@PaulAngus
Copy link
Member Author

request docbuild

@ACSGitBot
Copy link

Your request had been received, i'll go and build the documentation and check the output log for errors.

This shouldn't take long.

@ACSGitBot
Copy link

Build finished. You can review it at: https://acs-www.shapeblue.com/docs/WIP-PROOFING/pr131

Build Log Output:

No log errors found to report.

@PaulAngus PaulAngus requested a review from andrijapanicsb May 29, 2020 10:25
@andrijapanicsb andrijapanicsb self-requested a review May 29, 2020 14:32
Copy link
Contributor

@andrijapanicsb andrijapanicsb left a comment

Choose a reason for hiding this comment

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

minor edits required.

@andrijapanicsb
Copy link
Contributor

I like the green colour though...

also be an issue on Shared Networks which are using Internet routable/public (i.e. non-RFC1918)
IP addresses as the DNS service is then queriable from the public internet at large. A new
global setting "expose.dns.externally" has been added (with a default value of "true" in
order to keep backward compatibility) which limits the source of DNS queries to hosts on the
Copy link
Contributor

@andrijapanicsb andrijapanicsb May 29, 2020

Choose a reason for hiding this comment

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

Suggested change
order to keep backward compatibility) which limits the source of DNS queries to hosts on the
order to keep backward compatibility) which does not limit the source of DNS queries to hosts on the

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I see your point, i've reworded to be clearer.

@yadvr yadvr added this to the 4.14.0.0 milestone Jun 5, 2020
@yadvr yadvr merged commit fb9c7d0 into apache:4.14 Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants