Skip to content

Updated Bootstrap Script Changes to support /connection endpoint changes - #61

Open
pacific-ring wants to merge 3 commits into
mainfrom
fix-bootstrap-script
Open

pacific-ring wants to merge 3 commits into
mainfrom
fix-bootstrap-script

Conversation

@pacific-ring

@pacific-ring pacific-ring commented Aug 25, 2026 •

Copy link
Copy Markdown
  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

📋 Changes

Fix: adopt Auth0's new /connections/{id}/clients endpoint

  • Stop sending the deprecated enabled_clients field in connection create/update bodies (Auth0 now rejects it with a 400).
  • Add getConnectionEnabledClients() — reads the paginated /connections/{id}/clients sub-resource; returns null (unknown/missing scope) vs [] (none enabled).
  • Add ensureClientEnabledOnConnection() — enables the app via additive PATCH and verifies by re-reading (PATCH returns 204).
  • Make checkDatabaseConnectionChanges() async; merge passkey options instead of overwriting.

Feature: configurable application name

  • Add --client-name= flag / AUTH0_CLIENT_NAME env / interactive prompt (resolveClientName, validateClientName).
  • Replace hardcoded CLIENT_NAME with DEFAULT_CLIENT_NAME + setClientName/getClientName module state.
  • Add promptWithUser() helper (free-text prompt with default + non-TTY fallback).

@pacific-ring
pacific-ring requested a review from a team as a code owner August 25, 2026 14:30

@SurajThotakura SurajThotakura left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Add back the NOTICE.md

Comment thread NOTICE.md
@@ -1,57 +0,0 @@
================================================================================

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@pacific-ring why are we removing the NOTICE.md file. Can you check if this is by mistake?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Good catch — that removal was unintentional. I've restored NOTICE.md in e80b903.

Restore the third-party software NOTICE.md that was inadvertently removed
on this branch, per PR review feedback.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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