search

fortify / helm3-charts

Fortify Helm Charts to automate deployment of Software Security Center (SSC), ScanCentral SAST and ScanCentral DAST to a Kubernetes
MIT License
11 stars 10 forks source link

Fortify license variable incorrectly set in SAST helm chart secret if not installed without --set-file argument #15

Closed KyleJonas closed 11 months ago

KyleJonas commented 11 months ago

The fortify.license variable in charts/scancentral-sast/templates/secret.yaml is not being set correctly if you do not install SAST without the Helm argument --set-file=fortifyLicense=<FORTIFY_LICENSE_PATH> .

.Values.fortifyLicense should be .Values.secrets.fortifyLicense based on the format of the SAST values file.

Corrected line in the SAST secret.yaml

fortify.license: {{ required "fortifyLicense value is required. (Tip) Use \"helm install/upgrade\" argument --set-file=fortifyLicense=<FORTIFY_LICENSE_PATH> ." .Values.secrets.fortifyLicense | b64enc | quote }}
fbordallo commented 11 months ago

It has been corrected at c6a236c . Thank you for your help.