Wang1rrr opened a new pull request, #6998:
URL: https://github.com/apache/rocketmq-dashboard/pull/6998

   ## Summary
   
   Fixes #6997.
   
   - Share the path-only API base normalization between configuration and the 
OpenAI-compatible client.
   - Insert endpoint paths before query/fragment suffixes, preserving their raw 
encoding and values.
   - Validate the path rather than interpreting a query value ending in 
`/chat/completions` as an endpoint path.
   - Exercise model listing, complete and stream calls against a local HTTP 
server, plus configuration save/read round trips.
   
   ## Validation
   
   `mvn -B -ntp -Dtest=OpenAiCompatibleLlmClientTest,LlmConfigServiceTest test` 
on Java 21: **67 tests passed**, Checkstyle passed.
   
   Before the fix, five newly added regression cases failed: wrong HTTP request 
paths and mutated/un-normalized saved URLs. Final coverage includes plain and 
full endpoint bases, with and without fragments, plus escaped query values. The 
full backend suite was not run locally.
   
   AI-assisted implementation; the changes and regression output were reviewed 
before submission.
   
   


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