Skip to content

Improve SAML Getting Started docs#605

Merged
monfresh merged 2 commits intomainfrom
update-cert-docs
Apr 20, 2026
Merged

Improve SAML Getting Started docs#605
monfresh merged 2 commits intomainfrom
update-cert-docs

Conversation

@monfresh
Copy link
Copy Markdown
Contributor

Partners are sometimes confused about the difference between our cert that we
update every year, and their certs that they need to rotate.

This explains the difference between the two, and the recommended ways for partners to update our cert, as well as instructions for updating their certs.

Partners are sometimes confused about the difference between our cert that we
update every year, and their certs that they need to rotate.

This explains the difference between the two, and the recommended ways for partners to update our cert, as well as instructions for updating their certs.
@monfresh monfresh requested review from ajfarkas and nprimak March 31, 2026 18:32
Comment thread _pages/saml/getting-started.md Outdated

The easiest and recommended way is via our metadata endpoint. Consistent with the [SAML metadata specification](https://docs.oasis-open.org/security/saml/v2.0/saml-metadata-2.0-os.pdf){:class="usa-link--external"}, Login.gov's metadata for our sandbox environment is available at [https://idp.int.identitysandbox.gov/api/saml/metadata{{ site.data.saml.year.current }}](https://idp.int.identitysandbox.gov/api/saml/metadata{{ site.data.saml.year.current }}). Our production metadata endpoint is available at [https://secure.login.gov/api/saml/metadata{{ site.data.saml.year.current }}](https://idp.int.identitysandbox.gov/api/saml/metadata{{ site.data.saml.year.current }}).

This means that you don't have to manually copy our certificate from this page and then upload it to your systems once a year, during our [Annual Certificate Rotation](#annual-certificate-rotation). Instead, you can simply update the year in the metadata URL.
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.

A suggestion to clean it up a bit:

This means you don't have to manually copy our certificate from this page and upload it to your systems once a year during our [Annual Certificate Rotation](#annual-certificate-rotation). Instead, simply update the year in the metadata URL

Comment thread _pages/saml/getting-started.md Outdated

This means that you don't have to manually copy our certificate from this page and then upload it to your systems once a year, during our [Annual Certificate Rotation](#annual-certificate-rotation). Instead, you can simply update the year in the metadata URL.

If your systems don't support ingesting the metadata via a URL, there typically is an option to upload a file. In that case, you would visit our metadata endpoint in your browser, then right-click anywhere inside the page, and select "Save As". The default filename will be metadata{{ site.data.saml.year.current }}.xml
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.

If it makes sense -- A suggestion to update "there typically is an option to upload a file" to "an option to upload a file is typically available" so it reads:

If your systems don't support ingesting the metadata via a URL, an option to upload a file is typically available.

Comment thread _pages/saml/getting-started.md Outdated

If your systems don't support ingesting the metadata via a URL, there typically is an option to upload a file. In that case, you would visit our metadata endpoint in your browser, then right-click anywhere inside the page, and select "Save As". The default filename will be metadata{{ site.data.saml.year.current }}.xml

If there aren't any metadata options available to you, you will need to manually copy the certificate from the next section, then either paste it on your end, or save it to a file with a .crt extension, then upload it on your end so your app can access it.
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.

Looks like ".crt" is marked as a regular text. Should it be marked in .crt inline code text?

@mmagsa
Copy link
Copy Markdown
Contributor

mmagsa commented Apr 1, 2026

Thanks for adding all this @monfresh! I'm a concerned that adding so much text makes this section a bit unwieldy, especially for a "getting started" / overview section. Is there some way we could shorten it, or give just a brief summary, with details in an accordion or behind a link? (cc @nickttng)

Comment on lines -14 to -15
- text: Metadata
href: "/saml/getting-started/#metadata"
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.

Changing headers is always dangerous. If we're linking to this section from somewhere, the link will break. Should we keep the section, or have you checked to make sure this won't break anything?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We are not linking to it from the dev docs site or from the login.gov site. Are there other places that I should check?

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 Partner Portal, and IdP protocol error messages are candidates. I just checked and didn't turn up anything there.

@monfresh
Copy link
Copy Markdown
Contributor Author

@nickttng @mmagsa Thanks for the feedback. I've added the Metadata header back in, and I've made the docs more concise. Can you review again please?

@nickttng
Copy link
Copy Markdown
Contributor

@nickttng @mmagsa Thanks for the feedback. I've added the Metadata header back in, and I've made the docs more concise. Can you review again please?

This looks good to me. Deferring to mmagsa as the PR approver.

@monfresh monfresh requested a review from mmagsa April 15, 2026 14:47
Copy link
Copy Markdown
Contributor

@mmagsa mmagsa left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@monfresh monfresh merged commit 971e820 into main Apr 20, 2026
9 checks passed
@monfresh monfresh deleted the update-cert-docs branch April 20, 2026 20:18
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