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

Use entire chain for the server cert #317

Merged
merged 4 commits into from Mar 27, 2024
Merged

Use entire chain for the server cert #317

merged 4 commits into from Mar 27, 2024

Conversation

sed-i
Copy link
Contributor

@sed-i sed-i commented Mar 26, 2024

Issue

Certificates generated by the charm don't include intermediates: canonical/httprequest-lego-k8s-operator#93.

Solution

Fetch-lib and use the new .chain_as_pem method.

In tandem with:

@PietroPasotti
Copy link
Collaborator

re the static error: issue is that we can't tell the static checker that in that code block we're working with pydantic v1.
So let's just ignore any error in there.

@sed-i sed-i marked this pull request as ready for review March 26, 2024 15:12
@PietroPasotti
Copy link
Collaborator

Given the specific upgrade instructions, should we take precautions as we publish this to edge, or do we go ahead and merge?

@sed-i sed-i force-pushed the feature/chain-pem branch 2 times, most recently from f479d12 to f138026 Compare March 27, 2024 15:52
@sed-i sed-i merged commit 8bdb61b into main Mar 27, 2024
13 checks passed
@sed-i sed-i deleted the feature/chain-pem branch March 27, 2024 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants