Skip to content

docs: correct not_enforced now that the default is deny - #87

Merged
lakhansamani merged 2 commits into
mainfrom
fix/not-enforced-now-denies
Aug 13, 2026
Merged

docs: correct not_enforced now that the default is deny#87
lakhansamani merged 2 commits into
mainfrom
fix/not-enforced-now-denies

Conversation

@lakhansamani

Copy link
Copy Markdown
Contributor

--fga-allow-unconstrained-agents changed the default: a delegated check against a model with no type agent is now denied rather than authorized as the delegating user alone.

agent-identity.md was updated at the top but not further down, so it contradicted itself — the callout says "is now denied", the observability table two screens later still said "authorized as the user alone". metrics-monitoring.md carried the same stale line.

Both now describe what not_enforced actually means, which depends on the flag:

  • unset — the check was denied. A misconfiguration users will report as a broken integration; this counter says why.
  • set — authorized as the delegating user alone, agent unconstrained. Silent by construction, so treat a non-zero rate as exposure with a clock on it.

Same remedy either way: add type agent and grant the agents.

Also fixes a broken anchor on /core/mcp (#self-registering-clients#self-registering-clients-cimd-vs-dcr) that was failing the build's anchor check on main — unrelated to the above, separate commit.

Verified: npm run build clean, zero broken links or anchors.

agent-identity.md contradicted itself: the callout at the top says a
delegated check against a model with no `type agent` is now denied, while
the observability table two screens down still said it is "authorized as
the user alone". metrics-monitoring.md carried the same stale line.

Both now describe what the outcome actually means, which depends on
--fga-allow-unconstrained-agents: unset it is a denial (a
misconfiguration users will report as a broken integration), set it is
the old unconstrained behaviour (exposure with a clock on it). Same
remedy either way.
The heading is 'Self-registering clients: CIMD vs DCR', so the slug
carries the suffix. Broke the build's anchor check on main.
@netlify

netlify Bot commented Aug 13, 2026

Copy link
Copy Markdown

Deploy Preview for authorizerdev-docs ready!

Name Link
🔨 Latest commit 4123f0a
🔍 Latest deploy log https://app.netlify.com/projects/authorizerdev-docs/deploys/6a7dcad218af4e0008335600
😎 Deploy Preview https://deploy-preview-87--authorizerdev-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@lakhansamani
lakhansamani merged commit 66f6cf7 into main Aug 13, 2026
4 checks passed
@lakhansamani
lakhansamani deleted the fix/not-enforced-now-denies branch August 13, 2026 14:09
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.

1 participant