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

   ## What changed
   
   - validate non-empty LLM base URLs before general settings are persisted
   - reject cloud metadata and other disallowed hosts through the shared 
`UrlHostGuard`
   - preserve loopback URLs for local LLM gateways
   - add regression coverage for both the blocked metadata endpoint and the 
allowed local endpoint
   
   ## Why
   
   The dedicated LLM configuration endpoint validates its base URL, but the 
general-settings save path persisted the same value without that guard. A 
metadata endpoint could therefore be stored through 
`/api/settings/general/save` and later used by the AI runtime.
   
   Fixes #2036.
   
   ## Verification
   
   - `mvn 
-Dtest=SettingsServiceTest,LlmConfigServiceTest,SettingsControllerTest test` — 
79 tests passed
   - `mvn test` — 1056 tests passed
   - Checkstyle — 0 violations
   - `git diff --check` — passed
   
   Signed-off-by: youngkermit8-coder <[email protected]>


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