Mark auth_client_secret variables as sensitive in terraform

[JC-wk]

Resolves #4735

What is being addressed

auth_client_secret leaks in log files if a deployment fails

How is this addressed

• Mark variable as sensitive in terraform
• Updated CHANGELOG.md
• Increment template versions
• Added sensitive var test

[github-actions]

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 87e5327.

♻️ This comment has been updated with latest results.

[JC-wk]

@marrobi I created a test which works locally but it doesn't seem like it's able to see the templates directory as it's current directory is api/. in the GitHub pipeline.
I can remove the test from the PR and create a separate issue if you think that is better to get this one merged sooner?
Happy to take suggestions on what to do with the testing.

[marrobi]

@marrobi I created a test which works locally but it doesn't seem like it's able to see the templates directory as it's current directory is api/. in the GitHub pipeline. I can remove the test from the PR and create a separate issue if you think that is better to get this one merged sooner? Happy to take suggestions on what to do with the testing.

It shoudl probably be called as part of: https://github.com/marrobi/AzureTRE/blob/e009b2497eb106d08018c86b86bbb0d5bf0d6831/Makefile#L253C1-L262

I'm happy with a templates/tests directory and tests get executed prior to bundle build.

@tamirkamara @jonnyry thoughts?

[JC-wk]

I removed the test and created #4737 to allow discussion @marrobi @tamirkamara @jonnyry so this one can get sorted more quickly.

[marrobi]

LGTM

[marrobi]

/test-extended

[github-actions]

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/19171576697 (with refid 2e041005)

(in response to this comment from @marrobi)

[marrobi]

/test 06785b2

[github-actions]

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/19466866239 (with refid 2e041005)

(in response to this comment from @marrobi)

[marrobi]

/test-force-approve 87e5327

Passed: https://github.com/microsoft/AzureTRE/actions/runs/19466866239

[github-actions]

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 87e5327)

(in response to this comment from @marrobi)