Re: [PR] Update ControllerAdminApiApplication.java [pinot]

2024-05-10 Thread via GitHub
sar12332111 closed pull request #13132: Update ControllerAdminApiApplication.java URL: https://github.com/apache/pinot/pull/13132 -- 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

Re: [PR] Update ControllerAdminApiApplication.java [pinot]

2024-05-10 Thread via GitHub
sar12332111 commented on code in PR #13132: URL: https://github.com/apache/pinot/pull/13132#discussion_r1597305755 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java: ## @@ -149,4 +140,20 @@ public void filter(ContainerRequestCon

Re: [PR] Update ControllerAdminApiApplication.java [pinot]

2024-05-10 Thread via GitHub
codecov-commenter commented on PR #13132: URL: https://github.com/apache/pinot/pull/13132#issuecomment-2105336499 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/13132?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

Re: [PR] Update ControllerAdminApiApplication.java [pinot]

2024-05-10 Thread via GitHub
Jackie-Jiang commented on code in PR #13132: URL: https://github.com/apache/pinot/pull/13132#discussion_r1597250043 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java: ## @@ -149,4 +140,20 @@ public void filter(ContainerRequestCo

[PR] Update ControllerAdminApiApplication.java [pinot]

2024-05-10 Thread via GitHub
sar12332111 opened a new pull request, #13132: URL: https://github.com/apache/pinot/pull/13132 Secure static file handling to prevent path traversal attacks - Implemented a new SafeStaticHttpHandler - Configured the ControllerAdminApiApplication - Modified the setupSwagger meth