Skip to content

Validate CN and/or SAN in certificate matches external dns zone during RSS. #4045

Closed

Description

@luqmana
No description provided.

Activity

  1. augustuswm commented on Sep 7, 2023

    @augustuswm
    Contributor

    This would be useful as another preflight tool that could be run once a cert has been uploaded and the external_dns_zone_name value has been set.

  2. luqmana commented on Sep 7, 2023

    @luqmana
    ContributorAuthor

    Yup, we already do some basic validation like making sure the cert and key match up. So good to catch this earlier as well.

  3. leftwo commented on Sep 7, 2023

    @leftwo
    Contributor

    Yeah, there is only so much we can do but any additional verification we can do before running RSS would be great.

  4. jgallagher commented on Sep 7, 2023

    @jgallagher
    Contributor

    #3436 has a bunch of extra checks that we can do at wicketd-upload time, but it fell off my radar. I'll try to get back to it soon.

  5. luqmana commented on Sep 7, 2023

    @luqmana
    ContributorAuthor

    Looking a lil more, we should probably do this not just during RSS but at Silo & Certificate creation in general. You can easily run into this just by creating a silo and giving it a mismatched cert.

  6. added this to the MVP milestone on Sep 14, 2023
  7. added a commit that references this issue on Sep 20, 2023
    373ddbd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions