geosolutions-it / charts

HELM Charts by GeoSolutions
Other
3 stars 6 forks source link

Typos in JAVA_OPTS #20

Closed camuffo closed 2 months ago

camuffo commented 3 months ago

There are small typos in the JAVA_OPTS value. https://github.com/geosolutions-it/charts/blob/c1b3852a156ac4450deaf12874708a6814adb0c9/geoserver/latest/templates/statefulset.yaml#L142-L144

-DGEOSERVER_CSRF_DISABLED=$(GEOSERVER_CSRF_DISABLED)\" should be -DGEOSERVER_CSRF_DISABLED=\"$(GEOSERVER_CSRF_DISABLED)\"

-DGEOSERVER_DATA_DIR=$(GEOSERVER_DATA_DIR)\" should be -DGEOSERVER_DATA_DIR=\"$(GEOSERVER_DATA_DIR)\"

-DGEOSERVER_AUDIT_PATH=$\"(GEOSERVER_AUDIT_PATH)\" should be -DGEOSERVER_AUDIT_PATH=\"$(GEOSERVER_AUDIT_PATH)\"

camuffo commented 3 months ago

PR https://github.com/geosolutions-it/charts/pull/21

randomorder commented 2 months ago

thanks @camuffo