-
Notifications
You must be signed in to change notification settings - Fork 190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AI PRP: Request Flyte Exposed UI & API #425 #528
base: master
Are you sure you want to change the base?
Conversation
Added Flyte Console Detector with RCE
Hi @tooryx , Can you review the code provide comments ? |
Hi @hayageek, Please be patient, a reviewer will pick-up your PR when they get through the queue. ~tooryx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @hayageek, thank you for your contribution.
You can find some style issues to address down below.
...xposed_console/src/main/java/com/google/tsunami/plugins/rce/ExposedFlyteConsoleDetector.java
Outdated
Show resolved
Hide resolved
...xposed_console/src/main/java/com/google/tsunami/plugins/rce/ExposedFlyteConsoleDetector.java
Outdated
Show resolved
Hide resolved
...xposed_console/src/main/java/com/google/tsunami/plugins/rce/ExposedFlyteConsoleDetector.java
Outdated
Show resolved
Hide resolved
...xposed_console/src/main/java/com/google/tsunami/plugins/rce/ExposedFlyteConsoleDetector.java
Outdated
Show resolved
Hide resolved
...ors/flyte_exposed_console/src/main/java/com/google/tsunami/plugins/rce/FlyteProtoClient.java
Outdated
Show resolved
Hide resolved
...ors/flyte_exposed_console/src/main/java/com/google/tsunami/plugins/rce/FlyteProtoClient.java
Outdated
Show resolved
Hide resolved
...ors/flyte_exposed_console/src/main/java/com/google/tsunami/plugins/rce/FlyteProtoClient.java
Show resolved
Hide resolved
...xposed_console/src/main/java/com/google/tsunami/plugins/rce/ExposedFlyteConsoleDetector.java
Show resolved
Hide resolved
@leonardo-doyensec , |
@leonardo-doyensec @tooryx |
Hi @leonardo-doyensec, |
Hi @tooryx @leonardo-doyensec |
Hi @hayageek, Sorry, I know that this is frustrating. We are merging things in reverse chronological order and doing our best, but we sometimes have competing priorities as you can imagine. That being said, I am not sure how pausing accepting pull requests would help? I actually think it would make things even worse. ~tooryx |
Hello @hayageek, |
@leonardo-doyensec , Could you share the commands you're using to run both the callback server and the Tsunami scanner? It appears there might be an issue with the callback server's IP address. |
Hi @tooryx ,
This is the PR for detecting RCE in exposed Flyte Console. Related issue is: #425
Also, created a PR in
security-testbeds
to setup the Flyte Console google/security-testbeds#84Flyte supports only gRPC APIs, and I have utilized the
flyte-java
SDK to make these API calls. Here's how the RCE vulnerability can be exploited in an exposed Flyte Console:registerProject
API.createTask
API.createExecution
API. Note that thecreateExecution
call may fail initially becausecreateTask
configures certain aspects in the Kubernetes cluster. In such cases, multiple attempts may be necessary. Once successfully executed, the Task spins up a Docker image and performs RCE within the POD.