Skip to content

Conversation

alexmatson-da
Copy link
Contributor

@alexmatson-da alexmatson-da commented Oct 13, 2025

Fixes #244

@alexmatson-da alexmatson-da force-pushed the alex/devnet-auth0-network branch from 2c062d4 to d40a055 Compare October 15, 2025 15:46
@alexmatson-da alexmatson-da marked this pull request as ready for review October 15, 2025 15:46
@alexmatson-da alexmatson-da requested review from a team as code owners October 15, 2025 15:46
@alexmatson-da alexmatson-da force-pushed the alex/devnet-auth0-network branch from d40a055 to 096336d Compare October 15, 2025 15:47
@alexmatson-da alexmatson-da force-pushed the alex/devnet-auth0-network branch from 096336d to ed6f9d8 Compare October 15, 2025 15:53
@alexmatson-da alexmatson-da enabled auto-merge (squash) October 15, 2025 15:53
clientId: network.auth.admin.clientId,
clientSecret: network.auth.admin.clientSecret,
scope: network.auth.scope,
scope: 'daml_ledger_api',
Copy link
Contributor

Choose a reason for hiding this comment

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

Why the hard code here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

because you need different scopes for the implicit auth used on the login page vs the client credentials auth used for the admin token

Copy link
Contributor Author

Choose a reason for hiding this comment

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

(and the config file currently only sets the former)

Copy link
Contributor

Choose a reason for hiding this comment

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

we should adjust the config then. Otherwise we'll drag it along for too long

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