Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Allow additional SANS for web certificate #1303

Merged
merged 1 commit into from
Dec 24, 2024

Conversation

heytrav
Copy link
Contributor

@heytrav heytrav commented Dec 19, 2024

This makes it possible to include extra variants of the service-name that aren't captured by the {{ include "capsule.fullname" }} macro

Copy link

netlify bot commented Dec 19, 2024

Deploy Preview for capsule-documentation canceled.

Name Link
🔨 Latest commit c4164b7
🔍 Latest deploy log https://app.netlify.com/sites/capsule-documentation/deploys/676a5bfe08c0d700085588f8

@heytrav heytrav force-pushed the fix/webhook-cert-sans branch from 757ee28 to 8603f89 Compare December 19, 2024 06:46
@heytrav heytrav changed the title Allow additional SANS for web certificate feat: Allow additional SANS for web certificate Dec 19, 2024
@heytrav heytrav force-pushed the fix/webhook-cert-sans branch 3 times, most recently from 51f71ee to 14a93b0 Compare December 21, 2024 23:52
charts/capsule/README.md Outdated Show resolved Hide resolved
@prometherion
Copy link
Member

CI is very picky, please, check the failed actions: if you're struggling let me know and I'll edit your PR!

@heytrav heytrav force-pushed the fix/webhook-cert-sans branch from f2c2bd2 to 627b2c7 Compare December 24, 2024 06:49
@heytrav
Copy link
Contributor Author

heytrav commented Dec 24, 2024

CI is very picky, please, check the failed actions: if you're struggling let me know and I'll edit your PR!

I think squashing the extra commit might help.

@prometherion
Copy link
Member

Documentation outdated! (Run make helm-docs locally and commit) and we should be there (famous last works ™️ )

@heytrav heytrav force-pushed the fix/webhook-cert-sans branch from 627b2c7 to 552b1a1 Compare December 24, 2024 06:58
This makes it possible to include extra variants of the service-name
that aren't captured by the {{ include "capsule.fullname" }} macro

Signed-off-by: Travis Holton <[email protected]>

Update charts/capsule/README.md

Co-authored-by: Dario Tranchitella <[email protected]>
@heytrav heytrav force-pushed the fix/webhook-cert-sans branch from 552b1a1 to c4164b7 Compare December 24, 2024 07:00
Copy link
Member

@prometherion prometherion left a comment

Choose a reason for hiding this comment

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

Thanks for your first contribution, hope to get more in the future! 🎉

@prometherion prometherion merged commit ed9e1d4 into projectcapsule:main Dec 24, 2024
16 checks passed
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