Skip to content

Correct disable_auth default to true - #317

Merged
jonoirwinrsa merged 1 commit into
masterfrom
docs/disable-auth-default
Aug 17, 2026
Merged

Correct disable_auth default to true#317
jonoirwinrsa merged 1 commit into
masterfrom
docs/disable-auth-default

Conversation

@jonoirwinrsa

@jonoirwinrsa jonoirwinrsa commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

The TOML reference lists disable_auth as defaulting to false, and the REST API page says functions are "accessible via authenticated POST requests by default". Both are wrong: an app deployed with no disable_auth in cerebrium.toml has authentication disabled, so its endpoints are callable without a token.

  • Flip the table default to true
  • Add a warning under the table, since a flipped boolean in a table cell is easy to miss for a setting that decides whether endpoints are public
  • Fix the contradicting "authenticated by default" sentence on the REST API page

The "Complete Example" block keeps its explicit disable_auth = false — that is a chosen value, not a default claim.

Verified the deployed default against the platform's app-create API. prettier --check passes on both files.

🤖 Generated with Claude Code

@mintlify

mintlify Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
cerebrium 🟢 Ready View Preview Aug 17, 2026, 9:02 PM

Deploying an app with no disable_auth in cerebrium.toml leaves
authentication disabled, so its endpoints are callable without a token.
Both the TOML reference table and the REST API page claimed the opposite.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@jonoirwinrsa
jonoirwinrsa force-pushed the docs/disable-auth-default branch from 291f0d8 to 4b69c63 Compare August 17, 2026 21:05
@jonoirwinrsa
jonoirwinrsa merged commit f758bb1 into master Aug 17, 2026
2 checks passed
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.

2 participants