Closed ershad-paladin closed 1 month ago
The updates to the PublicAccessforConfiguredPort
class enhance port validation logic for Azure resources. The method formerly focused on PostgreSQL server access has been broadened to validate public ports more generally, with added priority checks for rules. Improvements also include better handling of port ranges and clearer help text, refining the functionality and user experience in managing access rules.
File Path | Change Summary |
---|---|
.../NSGRule/PublicAccessforConfiguredPort.java |
Renamed validatePostgresqlServerAccess to validatePublicPorts , updated logic for priority checks on access rules, enhanced port range validation, and revised help text. |
🐰 In fields of code where rabbits hop,
New rules for ports, we’ll never stop!
With priorities clear and ranges neat,
Our access checks are now a treat!
So bounce along, let’s celebrate,
Validating ports, it’s simply great! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Failed conditions
B Maintainability Rating on New Code (required ≥ A)
See analysis details on SonarCloud
Catch issues before they fail your Quality Gate with our IDE extension SonarLint
Description
Problem
Solution
Fixes # (issue if any)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Tested screenshots: Nsg config:
Before fix:
After fix:
Checklist:
Other Information:
List any documentation updates that are needed for the Wiki
Summary by CodeRabbit
New Features
Documentation