Skip to content

(PUP-11120) Readd support for trusted_oid_mapping_file #8646

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

Merged
merged 2 commits into from
Jun 22, 2021

Conversation

joshcooper
Copy link
Contributor

Commit 68a10c6 allowed puppet cert print to translate custom oids into their
respective long name. This functionality was lost during the move to puppet ssl show and this commit readds it.

Prior to this commit, we would see:

$ puppet ssl show | grep -A2 X509v3
    X509v3 extensions:
        1.3.6.1.4.1.34380.1.2.1.1:
            ..somevalue

With this commit, the custom oid is translated to its long name:

    X509v3 extensions:
        Long name:
            ..somevalue

@joshcooper joshcooper requested review from a team June 17, 2021 15:24
@joshcooper joshcooper force-pushed the custom_oid_11120 branch 5 times, most recently from e62ded5 to 90bd36e Compare June 17, 2021 16:58
Update the rake gen_cert_fixtures task to generate a cert & key with an
extension containing a custom oid.
Commit 68a10c6 allowed `puppet cert print` to translate custom oids into their
respective long name. This functionality was lost during the move to `puppet ssl
show` and this commit readds it.

Prior to this commit, we would see:

    $ puppet ssl show | grep -A2 X509v3
        X509v3 extensions:
            1.3.6.1.4.1.34380.1.2.1.1:
                ..somevalue

With this commit, the custom oid is translated to its long name:

        X509v3 extensions:
            Long name:
                ..somevalue
@GabrielNagy GabrielNagy merged commit 8e1ded9 into puppetlabs:6.x Jun 22, 2021
@joshcooper joshcooper deleted the custom_oid_11120 branch June 22, 2021 13:45
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