-
Notifications
You must be signed in to change notification settings - Fork 61
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
Add proxy setup to prepare wrapper #3269
Conversation
lorriborri
commented
Jul 2, 2024
- closes Add Proxy for Prepare Wrapper #3260
48cbfb3
to
016c971
Compare
016c971
to
8d1120b
Compare
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.
Looking good 👍
I have some ideas, but most of them are more optional from my perspective. Please have a look at the comments.
sechub-pds-solutions/prepare/helm/pds-prepare/templates/deployment.yaml
Outdated
Show resolved
Hide resolved
...re/src/main/java/com/mercedesbenz/sechub/wrapper/prepare/cli/PrepareWrapperKeyConstants.java
Outdated
Show resolved
Hide resolved
...re/src/main/java/com/mercedesbenz/sechub/wrapper/prepare/cli/PrepareWrapperKeyConstants.java
Outdated
Show resolved
Hide resolved
...repare/src/main/java/com/mercedesbenz/sechub/wrapper/prepare/PrepareWrapperProxySupport.java
Outdated
Show resolved
Hide resolved
...repare/src/main/java/com/mercedesbenz/sechub/wrapper/prepare/PrepareWrapperProxySupport.java
Outdated
Show resolved
Hide resolved
4c9c393
to
2c80e42
Compare
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.
Looking good 👍
I just found a typo in the env file.
c3fede8
to
1b98895
Compare