AppSec Deployment does not respect .Values.tls.enabled

[timtrense-leadec]

When setting tls.enabled: true, the lapi will provide an https endpoint at 8080.

If I see it correctly, the deployment of the appsec component does not respect that when constructing the target URL.

Instead, http:// is hardcoded. Also, no env vars for the CLIENT_CERT_FILE/CLIENT_KEY_FILE/CACERT_FILE get set.

Do I see that correctly that the appsec component currently does not support tls.enabled or am I missing something?

PS Besides, great work in providing a helm chart for CrowdSec! Really would like to use it, once I can get AppSec up.

[github-actions]

@timtrense-leadec: Thanks for opening an issue, it is currently awaiting triage.

If you haven't already, please provide the following information:

• kind : bug, enhancementor documentation
• area : agent, appsec, configuration, cscli, local-api

In the meantime, you can:

1. Check Crowdsec Documentation to see if your issue can be self resolved.
2. You can also join our Discord.
3. Check Releases to make sure your agent is on the latest version.

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the forked project rr404/oss-governance-bot repository.

[github-actions]

@timtrense-leadec: There are no 'kind' label on this issue. You need a 'kind' label to start the triage process.

• /kind bug
• /kind documentation
• /kind enhancement

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the forked project rr404/oss-governance-bot repository.

[timtrense-leadec]

/kind bug
/area appsec

[he2ss]

Hi @timtrense-leadec,

it's correct, I implemented the appsec in the helm chart without the tls for a first iteration.
It will be done asap.

[timtrense-leadec]

Hey @he2ss, thank you for the quick response. I didn't mean to rush you: just good to have the confirmation to have read the code correctly.
Really looking forward to that!