Skip to content
This repository was archived by the owner on Aug 24, 2026. It is now read-only.

Increase maximum number of alternative origins to 100 - #6248

Closed
aterga wants to merge 3 commits into
masterfrom
claude/increase-alternative-origins-limit
Closed

Increase maximum number of alternative origins to 100#6248
aterga wants to merge 3 commits into
masterfrom
claude/increase-alternative-origins-limit

Conversation

@aterga

@aterga aterga commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Internet Identity is raising the cap on entries in /.well-known/ii-alternative-origins from 10 to 100 (dfinity/internet-identity#4261). This updates the developer docs to match.

Both statements of the old limit in the alternative frontend origins guide are updated:

  • docs/building-apps/authentication/alternative-origins.mdx — the "Constraints" section and the "Requirements" bullet under the ii-alternative-origins example.

No other page needed changing. docs/references/ii-spec.md is a pointer to submodules/internetidentity/docs/ii-spec.mdx, so the spec page picks up the new limit from the Internet Identity repo when that submodule advances. The other "10" occurrences in the docs (cycles-wallet.mdx, ic-interface-spec.md) are about canister controllers and unrelated limits.

This should land together with, or after, the Internet Identity change — until that ships, listing more than 10 origins is still rejected.

Checklist from the template, for a text-only edit to an existing page:

  • Follows the developer docs style guide.
  • Follows the best practices and guidelines.
  • New documentation pages include document tags — n/a, no new pages.
  • New documentation pages include SEO keywords — n/a, no new pages.
  • New documents are in the .mdx file format — the edited page already is.
  • New documents are registered in /sidebars.js — n/a, page is already registered.
  • .github/CODEOWNERS updated — n/a, no new directories or documents.

Generated by Claude Code

Internet Identity raised the cap on entries in
`/.well-known/ii-alternative-origins` from 10 to 100. Update both places
in the alternative frontend origins guide that state the old limit.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Xb4gLgdkcBhNJdByGQhiLc
@github-actions github-actions Bot added the documentation Changes to Developer Docs label Aug 20, 2026
@aterga
aterga requested a lite review from Copilot August 20, 2026 14:57
@aterga
aterga marked this pull request as ready for review August 20, 2026 14:58
@aterga
aterga requested a review from a team as a code owner August 20, 2026 14:58
@aterga
aterga enabled auto-merge (squash) August 20, 2026 14:58

Copilot AI 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.

Pull request overview

Updates the developer documentation for Internet Identity (II) alternative frontend origins to reflect the increased maximum number of entries in /.well-known/ii-alternative-origins.

Changes:

  • Update the stated maximum alternative origins limit from 10 to 100 in the “Constraints” section.
  • Update the stated maximum limit from 10 to 100 in the ii-alternative-origins “Requirements” list.
Suppressed comments (1)

docs/building-apps/authentication/alternative-origins.mdx:65

  • Same as above: this requirement line reads like current runtime behavior, but the PR description notes that older Internet Identity deployments still enforce 10. Adding a short note here helps prevent confusion when developers try >10 entries before the rollout lands everywhere.
* Maximum of **100** alternative origins can be listed.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/building-apps/authentication/alternative-origins.mdx Outdated
The page stated the 100-origin limit as current behavior, but Internet
Identity deployments that predate the increase still enforce 10. Drop
the "Currently" framing and say which deployments the old limit applies
to, in both places that state the maximum.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Xb4gLgdkcBhNJdByGQhiLc
Comment thread docs/building-apps/authentication/alternative-origins.mdx Outdated
Requested in review. Both statements of the maximum go back to naming
the limit alone. The "Currently" framing stays dropped, so neither line
asserts the limit as present behavior on every deployment.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Xb4gLgdkcBhNJdByGQhiLc
@marc0olo

Copy link
Copy Markdown
Member

This repository has been replaced by dfinity/developer-docs, live at docs.internetcomputer.org, and is being archived, so no further merges will happen here. Closing this PR — if the change is still relevant, please open an equivalent PR against dfinity/developer-docs. Thanks for the contribution.

@marc0olo marc0olo closed this Aug 24, 2026
auto-merge was automatically disabled August 24, 2026 09:31

Pull request was closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

documentation Changes to Developer Docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants