youngkermit8-coder opened a new pull request, #1944:
URL: https://github.com/apache/rocketmq-dashboard/pull/1944

   ## Summary
   
   - resolve every DNS answer in the shared URL host guard
   - reject the host when any address violates the existing any-local, 
link-local, or loopback policy
   - preserve loopback support for local LLM gateways
   - add focused coverage for mixed-order DNS results and invalid address sets
   
   ## Root cause
   
   `UrlHostGuard.isAllowedHost` used `InetAddress.getByName`, which validates 
only one DNS answer. The HTTP client resolves the hostname independently, so a 
hostname with mixed public and link-local/cloud-metadata results could pass 
validation based on answer ordering and later connect to a blocked address.
   
   This is a follow-up to #1543/#1556. That change protects the separate 
data-source connection-test check; the newer shared guard used by data-source 
persistence, metrics queries, and LLM API bases still inspected one answer.
   
   ## Validation
   
   - Focused URL guard and affected service tests: 60 passed
   - Full backend suite: 1058 passed
   - Checkstyle: 0 violations
   - `git diff --check`: passed
   
   Closes #1943


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to