Skip to content

IP SAN requirement for CA cert used in SCO. - #1112

Merged
Chr1st0ph3rTurn3r merged 5 commits into
masterfrom
sco-ipsan-cert-requirement
Aug 13, 2026
Merged

IP SAN requirement for CA cert used in SCO.#1112
Chr1st0ph3rTurn3r merged 5 commits into
masterfrom
sco-ipsan-cert-requirement

Conversation

@Chr1st0ph3rTurn3r

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread docs/sec-conductor-onboard.md Outdated

The public certificate and CA certificate are configured on the conductor at the Authority level.

Secure Conductor Onboarding requires the use of an IP SAN (subject alternative name) for each router that is onboarding to the conductor. This requires that the certificate issued by the CA includes includes the IP SAN, which allows the conductor to perform validation and authentication.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does the IP SAN need to match any fields in the configuration relating to the router? If so, that would be worth mentioning here as well, unless that is otherwise fully covered in the related link.

@gregschrock gregschrock Aug 10, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The SAN(s) would need to match the conductor IPs that are configured in the mist redirect site. The API is /api/v1/sites/<site>/setting and the relevant subset of the content looks like this:

{
    "ssr": {
        "conductor_hosts": ["10.92.53.7", "10.92.54.7"],
        "conductor_token": "<sco-token>"
    }
}

The IP SAN would need to include those conductor_hosts IPs as they are what the router will use to connect to the conductor.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

My prior comment was specific to redirect. More generically, the relevant config would be the authority -> conductor-address list which specifies addresses routers should use to reach the conductor.

Comment thread docs/sec-conductor-onboard.md Outdated

The public certificate and CA certificate are configured on the conductor at the Authority level.

Secure Conductor Onboarding requires the use of an IP SAN (subject alternative name) for each router that is onboarding to the conductor. This requires that the certificate issued by the CA includes includes the IP SAN, which allows the conductor to perform validation and authentication.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"for each router" makes it sound like the SAN is required multiple times. But it's only a requirement for the conductor's certificate (more of a one-time thing). I'd say ...for routers to securely identify the conductor's identity. The second sentence would need a similar adjustment as the SAN is for router's authenticating the conductor, not vice verse.

I'd also call out that multiple IP SAN entries are required for HA conductors (one for each node).

madamsJuniper
madamsJuniper previously approved these changes Aug 10, 2026
@MichaelBaj
MichaelBaj self-requested a review August 10, 2026 18:29
MichaelBaj
MichaelBaj previously approved these changes Aug 10, 2026
madamsJuniper
madamsJuniper previously approved these changes Aug 10, 2026
gregschrock
gregschrock previously approved these changes Aug 11, 2026

@gregschrock gregschrock left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Chr1st0ph3rTurn3r
Chr1st0ph3rTurn3r merged commit e27d1e3 into master Aug 13, 2026
1 check passed
@Chr1st0ph3rTurn3r
Chr1st0ph3rTurn3r deleted the sco-ipsan-cert-requirement branch August 13, 2026 18:04
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.

4 participants