Skip to content

website/integrations: macmon NAC#17898

Merged
tanberry merged 6 commits intogoauthentik:mainfrom
nicedevil007:macmon-nac
Nov 14, 2025
Merged

website/integrations: macmon NAC#17898
tanberry merged 6 commits intogoauthentik:mainfrom
nicedevil007:macmon-nac

Conversation

@nicedevil007
Copy link
Contributor

@nicedevil007 nicedevil007 commented Nov 2, 2025 •
edited
Loading

Details

Keeping the doc momentum going with a guide for macmon NAC.

Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make docs)

@nicedevil007 nicedevil007 requested a review from a team as a code owner November 2, 2025 09:18
@netlify
Copy link

netlify bot commented Nov 2, 2025 •
edited
Loading

✅ Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 32aa1bb
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/6915fa99be1ea600081ed0db
😎 Deploy Preview https://deploy-preview-17898--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Nov 2, 2025 •
edited
Loading

✅ Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 32aa1bb
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/6915fa9b3c130a0008931833

@netlify
Copy link

netlify bot commented Nov 2, 2025 •
edited
Loading

✅ Deploy Preview for authentik-integrations ready!

Name Link
🔨 Latest commit 32aa1bb
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/6915fa9900baa9000880e500
😎 Deploy Preview https://deploy-preview-17898--authentik-integrations.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Nov 2, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.93%. Comparing base (9295d87) to head (32aa1bb).
⚠️ Report is 26 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #17898      +/-   ##
==========================================
- Coverage   92.97%   92.93%   -0.04%     
==========================================
  Files         869      869              
  Lines       48039    48039              
==========================================
- Hits        44662    44643      -19     
- Misses       3377     3396      +19
Flag Coverage Δ
e2e 45.23% <ø> (+<0.01%) ⬆️
integration 23.13% <ø> (-0.06%) ⬇️
unit 91.08% <ø> (+<0.01%) ⬆️
unit-migrate 91.13% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dominic-r dominic-r self-requested a review November 2, 2025 16:48
@dominic-r dominic-r added the area:docs Features or issues related to Docusaurus label Nov 2, 2025
@dewi-tik dewi-tik self-assigned this Nov 4, 2025
@dewi-tik
Copy link
Contributor

dewi-tik commented Nov 10, 2025

Thanks again @nicedevil007. Are you happy with the changes that I made?

@dewi-tik dewi-tik moved this from Todo to Needs review in authentik Core Nov 10, 2025
nicedevil007
nicedevil007 commented Nov 10, 2025
View reviewed changes
Copy link
Contributor Author

@nicedevil007 nicedevil007 left a comment

Choose a reason for hiding this comment

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

mhhh I know that this seems to be the case for most applications but not for macmon.

This documentation lists only the settings that you need to change from their default values. Be aware that any changes other than those explicitly mentioned in this guide could cause issues accessing your application.

You cannot disable the "local login" right now in macmon. So you are only able to missconfigure the SSO part and still be able to login with the local admin.

Is there a "standardized sentence" that we can add here?

@nicedevil007
Copy link
Contributor Author

nicedevil007 commented Nov 10, 2025

mhhh I know that this seems to be the case for most applications but not for macmon.

This documentation lists only the settings that you need to change from their default values. Be aware that any changes other than those explicitly mentioned in this guide could cause issues accessing your application.

You cannot disable the "local login" right now in macmon. So you are only able to missconfigure the SSO part and still be able to login with the local admin.

Is there a "standardized sentence" that we can add here?

screenshot here, upper part will always be visible.

image

this is still the case for the most recent version 6.5

@dewi-tik
Copy link
Contributor

dewi-tik commented Nov 11, 2025

mhhh I know that this seems to be the case for most applications but not for macmon.

This documentation lists only the settings that you need to change from their default values. Be aware that any changes other than those explicitly mentioned in this guide could cause issues accessing your application.

You cannot disable the "local login" right now in macmon. So you are only able to missconfigure the SSO part and still be able to login with the local admin.

Is there a "standardized sentence" that we can add here?

This is our standardized sentence. The "cause issues accessing your application" means via authentik. We use the same info box on all guides.

For the time being we can go with this version, but we can look to potentially change it on all guides with a different PR.

dewi-tik
dewi-tik approved these changes Nov 11, 2025
View reviewed changes
Copy link
Contributor

@dewi-tik dewi-tik left a comment

Choose a reason for hiding this comment

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

Thanks again!

@github-project-automation github-project-automation bot moved this from Needs review to In Progress in authentik Core Nov 11, 2025
@dominic-r dominic-r self-assigned this Nov 13, 2025
@dominic-r dominic-r added this to the Release 2025.12 milestone Nov 14, 2025
@dewi-tik dewi-tik moved this from In Progress to Needs review in authentik Core Nov 14, 2025
tanberry
tanberry approved these changes Nov 14, 2025
View reviewed changes
Copy link
Contributor

@tanberry tanberry left a comment

Choose a reason for hiding this comment

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

Thanks @nicedevil007 !

@github-project-automation github-project-automation bot moved this from Needs review to In Progress in authentik Core Nov 14, 2025
@tanberry tanberry merged commit 4c92eff into goauthentik:main Nov 14, 2025
96 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in authentik Core Nov 14, 2025
kensternberg-authentik added a commit that referenced this pull request Dec 11, 2025
@kensternberg-authentik
Merge branch 'main' into dev
63e1887 
* main:
  website/integrations: FortiMail (#17900)
  web/sfe: downgrade bootstrap that was accidentally upgraded (#18157)
  web: Fix ESBuild hanging process (#18162)
  website/integrations: macmon NAC (#17898)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanberry tanberry tanberry approved these changes

@dewi-tik dewi-tik dewi-tik approved these changes

@dominic-r dominic-r Awaiting requested review from dominic-r

Assignees

@tanberry tanberry

@dominic-r dominic-r

@dewi-tik dewi-tik

Labels

area:docs Features or issues related to Docusaurus

Projects

None yet

Milestone

Release 2025.12

Development

Successfully merging this pull request may close these issues.

4 participants

@nicedevil007 @dewi-tik @tanberry @dominic-r