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

CA Export Format #902

Merged
merged 1 commit into from
Apr 6, 2017
Merged

CA Export Format #902

merged 1 commit into from
Apr 6, 2017

Conversation

russjones
Copy link
Contributor

@russjones russjones commented Apr 5, 2017

Purpose

In #839 we corrected the User CA export format for OpenSSH comparability. However, as documented in #869 we did not update our own code that parses certificates, specifically for Trusted Clusters, to understand User CA in the correct authorized_keys format. In addition, we were not exporting the cluster name in the new format.

Implementation

  • When exporting a user certificate, include the cluster name and type in the comment field.
  • When parsing a certificate first determine the format, is it in authorized_keys format or known_hosts format. Based off that parse the certificates appropriately.
  • Added --compat=1.0 flag to tctl to export in the old format for legacy situations.

Related Issues

Fixes #869

@russjones russjones merged commit e05a5c3 into master Apr 6, 2017
@russjones russjones deleted the rjones/ca-format branch April 6, 2017 19:16
hatched pushed a commit to hatched/teleport-merge that referenced this pull request Nov 30, 2022
hatched pushed a commit that referenced this pull request Dec 20, 2022
hatched pushed a commit that referenced this pull request Feb 1, 2023
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