Skip to content

Conversation

d0ugal
Copy link
Contributor

@d0ugal d0ugal commented Aug 28, 2025

This builds on #1468 and adds support to http checks.

String interpolation is supported in the basic auth password field, bearer token and TLS fields.

If #1468 hasn't merged it is best to look at the individual commits as its included.

@d0ugal d0ugal requested a review from a team as a code owner August 28, 2025 13:21
@d0ugal d0ugal requested review from nadiamoe and Pokom and removed request for nadiamoe and Pokom August 28, 2025 13:21
@d0ugal d0ugal marked this pull request as draft August 28, 2025 13:21
@d0ugal d0ugal changed the base branch from main to feature/pr1-core-secret-infrastructure-and-interpolation August 28, 2025 13:40
@d0ugal d0ugal changed the base branch from feature/pr1-core-secret-infrastructure-and-interpolation to main August 28, 2025 13:41
@d0ugal d0ugal closed this Aug 28, 2025
@d0ugal d0ugal deleted the feature/pr2-http-prober-secret-integration branch August 28, 2025 13:47
@d0ugal d0ugal restored the feature/pr2-http-prober-secret-integration branch August 28, 2025 13:47
@d0ugal d0ugal reopened this Aug 28, 2025
@d0ugal d0ugal force-pushed the feature/pr2-http-prober-secret-integration branch from 31370e9 to b4fdd61 Compare August 28, 2025 14:28
@d0ugal d0ugal changed the title feat: integrate secret resolution into HTTP prober and agent components feat: use string interpolation in the http prober Aug 28, 2025
@d0ugal d0ugal force-pushed the feature/pr2-http-prober-secret-integration branch 2 times, most recently from 06488b9 to b050ec1 Compare August 28, 2025 15:25
@d0ugal d0ugal changed the title feat: use string interpolation in the http prober feat: add support for secrets in the http prober Aug 28, 2025
@d0ugal d0ugal changed the title feat: add support for secrets in the http prober feat: add support for secrets in http checks Aug 28, 2025
@d0ugal d0ugal marked this pull request as ready for review August 28, 2025 15:51
@d0ugal d0ugal requested a review from mem August 28, 2025 15:52
@d0ugal d0ugal force-pushed the feature/pr2-http-prober-secret-integration branch 5 times, most recently from 2f66c0b to 822cbe3 Compare September 4, 2025 07:48
@d0ugal d0ugal force-pushed the feature/pr2-http-prober-secret-integration branch 4 times, most recently from 6ad9c7a to afc0633 Compare September 5, 2025 15:19
@d0ugal d0ugal force-pushed the feature/pr2-http-prober-secret-integration branch from afc0633 to d87f053 Compare September 5, 2025 15:24
@d0ugal d0ugal force-pushed the feature/pr2-http-prober-secret-integration branch from d87f053 to 44e7a89 Compare September 5, 2025 15:27
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mem this is the file that I mentioned in our call

Copy link
Contributor

@mem mem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is good. I would hold back merging after the release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants