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

fix: Unset REGISTRY_ prefixed env vars #424

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

jimmidyson
Copy link
Contributor

The Docker registry that is embedded in mindthegap reads REGISTRY_ prefixed
environment variables, which could lead to unexpected behaviour depending on
the environment. This commit unsets these environment variables to ensure
consistent behaviour independent of environment.

Fixes D2IQ-97438.

@jimmidyson jimmidyson requested a review from cneth June 5, 2023 11:02
@github-actions github-actions bot added the fix label Jun 5, 2023
@jimmidyson jimmidyson enabled auto-merge (squash) June 5, 2023 11:03
@jimmidyson jimmidyson disabled auto-merge June 5, 2023 11:04
@jimmidyson jimmidyson enabled auto-merge (squash) June 5, 2023 11:04
@jimmidyson jimmidyson force-pushed the jimmi/D2IQ-97438-registry-envvars branch from 99f11df to d162d3e Compare June 5, 2023 11:04
The Docker registry that is embedded in mindthegap reads `REGISTRY_` prefixed
environment variables, which could lead to unexpected behaviour dependin on
the environment. This commit unsets these environment variables to ensure
consistent behaviour independent of environment.

Fixes D2IQ-97438.
@jimmidyson jimmidyson force-pushed the jimmi/D2IQ-97438-registry-envvars branch from d162d3e to ccaf0ae Compare June 5, 2023 11:14
Copy link

@cneth cneth left a comment

Choose a reason for hiding this comment

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

LGTM!

@jimmidyson jimmidyson disabled auto-merge June 5, 2023 14:47
@jimmidyson jimmidyson merged commit d48a2b0 into main Jun 5, 2023
@jimmidyson jimmidyson deleted the jimmi/D2IQ-97438-registry-envvars branch June 5, 2023 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants