Skip to content

Conversation

@dicagno
Copy link
Collaborator

@dicagno dicagno commented Jul 28, 2025

this fix is essential, as it enables teams, partners and customers to seamlessly grant required access level to the api key generated, without requiring additonal documentation or knowledge transfer

@dicagno dicagno requested a review from duynguyen July 28, 2025 13:49
}

// Make the POST request to update the site configuration
const postResponse = await fetch(siteConfigEndpoint, {
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we make the post to the /apiKey.json endpoint specifically, which is safer and wouldn't affect other site configs in any way?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We already do that to create the key, but then we need to enable it for operation, and this is done in the site config.

Copy link
Collaborator

Choose a reason for hiding this comment

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

no this is done on the access configuration, you can try /access/admin.json instead to limit the scope of the change.

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.

3 participants