Skip to content
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

fix: disable jersey wadl by default #7074

Merged
merged 1 commit into from Mar 28, 2024
Merged

fix: disable jersey wadl by default #7074

merged 1 commit into from Mar 28, 2024

Conversation

ytvnr
Copy link
Member

@ytvnr ytvnr commented Mar 28, 2024

@passionne
Copy link
Contributor

What do you think to add it also in helm parameter as default:
https://github.com/gravitee-io/gravitee-api-management/blob/master/helm/values.yaml#L556

      - name: JAVA_OPTS
        value: "-Djersey.config.allowSystemPropertiesProvider=true -Djersey.config.server.wadl.disableWadl=true"

?

@ytvnr
Copy link
Member Author

ytvnr commented Mar 28, 2024

@passionne I can, but not sure it will be useful as docker images are already running ./bin/gravitee
do you see a particular interest in doing that also in helm chart ?

@ytvnr ytvnr marked this pull request as draft March 28, 2024 08:22
@passionne
Copy link
Contributor

@passionne I can, but not sure it will be useful as docker images are already running ./bin/gravitee do you see a particular interest in doing that also in helm chart ?

Right ! I had in mind that this was mainly use for rpm and zip installation, I forgot the docker entrypoint.

@ytvnr ytvnr marked this pull request as ready for review March 28, 2024 13:21
@ytvnr ytvnr merged commit f9e22d4 into 4.3.x Mar 28, 2024
31 of 32 checks passed
@ytvnr ytvnr deleted the apim-4310-jersey-wadl branch March 28, 2024 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants