Skip to content

Conversation

rnandwal-f5
Copy link
Contributor

Proposed changes

Problem: The NGINX listen port restrictions section in the NGINXaaS for Azure documentation was missing some recent changes for blocked ports. It only mentioned "certain ports" without providing a comprehensive list of all blocked ports, making it difficult for users to understand exactly which ports they cannot use with the listen directive.

Solution: Updated the documentation to include a complete and detailed list of all blocked ports with explanatory descriptions. The changes include:

Added explicit enumeration of all currently blocked ports (19, 21, 70, 119, 49151, 49153, 5140, 50000, 54141, 54779)
Categorized ports by their purpose (Azure health probe restricted ports, Azure system reserved port, internal service ports)
Provided clear explanations for why each port or group of ports is blocked
Maintained the existing bullet point structure for consistency with the rest of the document
This ensures users have complete visibility into port restrictions and understand the technical reasons behind each limitation.

Testing: Verified that the documentation renders correctly in Markdown and that all port numbers and descriptions are accurate. Cross-referenced the port list with current NGINXaaS for Azure restrictions to ensure completeness.

Please focus on: Review the accuracy of the port descriptions and categorizations to ensure they align with the current NGINXaaS for Azure implementation. Confirm that the technical explanations are clear and helpful for users who encounter port conflict

Checklist

Before merging a pull request, run through this checklist and mark each as complete.

  • I have read the contributing guidelines
  • I have signed the F5 Contributor License Agreement (CLA)
  • I have rebased my branch onto main
  • I have ensured my PR is targeting the main branch and pulling from my branch from my own fork
  • I have ensured that the commit messages adhere to Conventional Commits
  • I have ensured that documentation content adheres to the style guide
  • If the change involves potentially sensitive changes1, I have assessed the possible impact
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • I have ensured that existing tests pass after adding my changes
  • If applicable, I have updated README.md

Footnotes

  1. 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.

@rnandwal-f5 rnandwal-f5 requested a review from a team as a code owner July 17, 2025 10:00
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginxaas-azure Issues related to NGINX as a Service for Azure labels Jul 17, 2025
Copy link

github-actions bot commented Jul 17, 2025

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@rnandwal-f5
Copy link
Contributor Author

I have hereby read the F5 CLA and agree to its terms

@JTorreG JTorreG merged commit f8e6299 into nginx:main Jul 17, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/nginxaas-azure Issues related to NGINX as a Service for Azure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants