Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lock while generating certificates or refreshing tokens #1349

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

pjcdawkins
Copy link
Collaborator

@pjcdawkins pjcdawkins commented Nov 24, 2023

This guards against most of the race condition bugs that occur while running
multiple instances of the CLI in parallel. It doesn't fix all of them.

src/Service/Api.php Outdated Show resolved Hide resolved
@pjcdawkins pjcdawkins force-pushed the auth-lock branch 4 times, most recently from 4e22f82 to b9c4945 Compare November 30, 2023 22:55
This guards against most of the race condition bugs that occur while running
multiple instances of the CLI in parallel. It doesn't fix all of them.

This behavior can be disabled using the environment variable
`{PREFIX}DISABLE_LOCKS`, or the config entry `api.disable_locks`.
@pjcdawkins pjcdawkins merged commit 76034a2 into platformsh:main Nov 30, 2023
1 check passed
@pjcdawkins pjcdawkins deleted the auth-lock branch November 30, 2023 23:04
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.

1 participant