userhimanshuverma opened a new pull request, #15728:
URL: https://github.com/apache/pinot/pull/15728

   ### **Issue:**
   
   Follow-up to 
[[#15706](https://github.com/apache/pinot/pull/15706)](https://github.com/apache/pinot/pull/15706)
 — fixes regression in Swagger UI link behavior introduced while adding reverse 
proxy (e.g., Apache Knox) compatibility.
   
   ### **Description:**
   
   This PR resolves a bug where the "Swagger REST API" link in the Pinot 
Controller UI redirects to the landing page (`/`) instead of loading the 
Swagger UI (`/help` or `/api`) when running a local setup (e.g., 
`localhost:9000`).
   
   The issue was caused by using a relative path (`./help`) in the navigation 
link, which breaks local routing in some setups. While this worked for Knox 
proxies, it broke the local development experience.
   
   ### **Changes Made:**
   
   * Updated the Swagger link in `Layout.tsx` to use an absolute path (`help`) 
instead of a relative one (`./help`).
   * Ensured the link works consistently from controller UI.
   
   
   ### **Impact:**
   
   * Restores expected Swagger UI functionality in local environments.
   * Improves robustness of Controller UI navigation.
   
   ### **Testing:**
   
   * Verified Swagger UI loads correctly via `http://localhost:9000/help`.
   * Checked that all other navigation items in the Controller UI work as 
expected.
   


-- 
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: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to