Skip to content

SAML Audience config update - #313

Merged
sunilarjun merged 5 commits into
neuvector:mainfrom
leotseng-suse:audience-config-update
Sep 24, 2026
Merged

sunilarjun merged 5 commits into
neuvector:mainfrom
leotseng-suse:audience-config-update

Conversation

@leotseng-suse

Copy link
Copy Markdown
Collaborator

No description provided.

leotseng-suse and others added 2 commits September 23, 2026 14:09
Rewrite the Okta guide around the current Okta Admin Console flow
(create app integration, configure SAML, attribute statements,
assignments, metadata details) and replace the Okta Classic and
2018-era NeuVector screenshots with current ones.

Document the audience URI on all three SAML pages: NeuVector derives it
from its SAML Redirect URI and does not expose it as a setting, so the
Okta Audience URI (SP Entity ID), the Azure Identifier (Entity ID) and
the AD FS relying party trust identifier all take that same value.

Also update the Azure and AD FS pages to the current NeuVector SAML
Settings screenshot and describe the Group Claim, Default Role and
"Add to Top" group role map that replaced the old Admin/Reader/CI role
map fields. The replaced screenshots exposed a console IP and port, an
Okta org URL, an internal AD FS FQDN and a signing certificate.

Addresses of consoles and IdP identifiers in the new screenshots are
replaced with placeholders.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
A user whose global role is none and who has no namespace role fails
authorization after a successful authentication, so state on the Okta,
Azure AD and AD FS pages that a role must come from either the default
role or the group role map.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Comment thread docs/08.integration/04.saml/04.saml.md Outdated
@sunilarjun
sunilarjun self-requested a review September 23, 2026 18:21

@sunilarjun sunilarjun 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.

Thanks for updating @leotseng-suse! I've left review comments below but only in the docs next version pages, the same updates need to be applied to the respective counterpart pages in the versioned_docs/version-5.6 pages.

Also noting from the related issue comment the changes are also in the 5.5 and 5.4 versions, do the 5.5/5.4 docs versions also need the changes applied there?

Comment thread docs/08.integration/04.saml/04.saml.md Outdated
Comment thread docs/08.integration/04.saml/04.saml.md Outdated
Comment thread docs/08.integration/04.saml/04.saml.md Outdated
Comment thread docs/08.integration/04.saml/04.saml.md Outdated
Comment thread docs/08.integration/04.saml/04.saml.md Outdated
Comment thread docs/08.integration/04.saml/04.saml.md Outdated
Comment thread docs/08.integration/05.msazure/05.msazure.md Outdated
Comment thread docs/08.integration/05.msazure/05.msazure.md Outdated
Comment thread docs/08.integration/05.msazure/05.msazure.md Outdated
Comment thread docs/08.integration/06.adfs/06.adfs.md Outdated
Okta: shift the headings up one level so the page starts at H2, split the
SAML Redirect URI prerequisite into a bullet with a sub-bullet, and merge
the two Configure SAML notes into one.

Azure AD and ADFS: switch the unordered lists to "-" bullets and bold the
"Add to Top" button name.
The audience URI behaviour and the role requirement apply to 5.5 and 5.4
as well, so bring their Okta, Azure AD and ADFS pages in line with 5.6.

Also retires the legacy screenshots that showed a real console address
and signing certificate (azure6-8.png, nv_adfs1.png, okta.png, saml.png,
saml1.png, setting.png) and fixes a "Kubernetess" typo in the ADFS page.
@leotseng-suse

leotseng-suse commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator Author

Thanks for updating @leotseng-suse! I've left review comments below but only in the docs next version pages, the same updates need to be applied to the respective counterpart pages in the versioned_docs/version-5.6 pages.

Also noting from the related issue comment the changes are also in the 5.5 and 5.4 versions, do the 5.5/5.4 docs versions also need the changes applied there?

@sunilarjun Yes, I also updated the 5.5 and 5.4 versions. Please help review them. Thank you.

Signed-off-by: Sunil Singh <sunil.singh@suse.com>

@sunilarjun sunilarjun 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.

@leotseng-suse Thank you for addressing the changes and updating v5.5/v5.4. I also did some updates to fix header levels, remove extra whitespace and extra line breaks in this reference commit. Noting I will update in product docs asap with the changes here.

@sunilarjun
sunilarjun merged commit ce763fe into neuvector:main Sep 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants