-
Notifications
You must be signed in to change notification settings - Fork 82
361 httptraffic #517
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
361 httptraffic #517
Conversation
…plifying longer sentences and enhancing readability.
🎉 Thank you for your contribution! It appears you have not yet signed the F5 Contributor License Agreement (CLA), which is required for your changes to be incorporated into an F5 Open Source Software (OSS) project. Please kindly read the F5 CLA and reply on a new comment with the following text to agree: I have hereby read the F5 CLA and agree to its terms 2 out of 3 committers have signed the CLA. |
I have hereby read the F5 CLA and agree to its terms |
recheck |
I have hereby read the F5 CLA and agree to its terms |
recheck |
content/nginx/admin-guide/security-controls/securing-http-traffic-upstream.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for your contributions!
While we're at the conference, I'd love to chat about this PR. We're doing an unconference session tomorrow (Tuesday) at 8:30 am
content/nginx/admin-guide/security-controls/securing-http-traffic-upstream.md
Outdated
Show resolved
Hide resolved
content/nginx/admin-guide/security-controls/securing-http-traffic-upstream.md
Outdated
Show resolved
Hide resolved
content/nginx/admin-guide/security-controls/securing-http-traffic-upstream.md
Outdated
Show resolved
Hide resolved
content/nginx/admin-guide/security-controls/securing-http-traffic-upstream.md
Outdated
Show resolved
Hide resolved
recheck |
Required contributors have signed the CLA |
Of course! It was a pleasure and a great opportunity to learn a bit about Git! I would love to chat about this PR. I will likely not be able to make the unconference session. Would it be ok if I found you sometime throughout the day and potentially coordinate through Slack? |
Hi @mjang! I am heading over to Revolution Hall in a few. Are you in the library where the unconference sessions are? |
Proposed changes
Problem: The page contained many long sentences and clauses that could be simplified to enhance readability.
Solution: I split up longer sentences into smaller sentences, and I also changed a list separated by commas into bullet points.
Testing: N/A
Please focus on (optional): I focused my edits on the top portion of the document except for the bullet point section toward the end of the document.
If this PR addresses an issue on GitHub, ensure that you link to it here:
Closes #361
Checklist
Before merging a pull request, run through this checklist and mark each as complete.
README.md
andCHANGELOG.md
Footnotes
Potentially sensitive changes include anything involving code, personally identify information (PII), live URLs or significant amounts of new or revised documentation. Please refer to our style guide for guidance about placeholder content. ↩