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

Update 07-workload-prerequisites.md with warning about PKCS formats #255

Merged
merged 4 commits into from
Nov 16, 2021

Conversation

scaswell-hirez
Copy link
Contributor

@scaswell-hirez scaswell-hirez commented Nov 8, 2021

Azure Key Vault will reject PEM files that contain private keys that are not in PKCS8 format.

Adding additional warning regarding accepted private key formats for Azure Key Vault.
@ckittel
Copy link
Member

ckittel commented Nov 10, 2021

Wow, good catch. @magrande, can you confirm that you missed that file when doing the Traefik update?

@scaswell-hirez, you'll need to sign the CLA if you would like this contribution committed.

@ckittel
Copy link
Member

ckittel commented Nov 10, 2021

@scaswell-hirez -- can you rebase your branch? This change that you proposed for the Traefik version has already been made. The other comment though didn't. Maybe you can rebase and reopen the PR with the updated title/description to match. Thanks.

Merge mspnp into hirez fork
@ckittel
Copy link
Member

ckittel commented Nov 16, 2021

Thanks for the rebase. If you want to update the title and description now to match the contents of this PR, that would be helpful.

@scaswell-hirez scaswell-hirez changed the title Update traefik.yaml to pull the latest version of traefik Update 07-workload-prerequisites.md with warning about PKCS formats Nov 16, 2021
07-workload-prerequisites.md Outdated Show resolved Hide resolved
Copy link
Member

@ckittel ckittel left a comment

Choose a reason for hiding this comment

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

Thanks!

@ckittel ckittel merged commit 43aef8b into mspnp:main Nov 16, 2021
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.

3 participants