Skip to content

fix: Re-enables default credentials for local development#48

Merged
tfelix merged 1 commit into
mainfrom
feature/fix-credential-defaults
Jun 30, 2026
Merged

fix: Re-enables default credentials for local development#48
tfelix merged 1 commit into
mainfrom
feature/fix-credential-defaults

Conversation

@tfelix

@tfelix tfelix commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

These credentials are important to start up a tf-runner or run-controller locally and therefore must not be removed until a proper solution is found.

@tfelix tfelix requested a review from malhussan June 29, 2026 09:35
@tfelix tfelix force-pushed the feature/fix-credential-defaults branch 2 times, most recently from bd63e4c to 19ffebe Compare June 29, 2026 09:38

@grubmeshi grubmeshi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ok, let's keep it consistent and remove it later once deprecated basic auth is really supposed to go away.

These credentials are important to start up a tf-runner or run-controller locally
and therefore must not be removed until a proper solution is found.
@tfelix tfelix force-pushed the feature/fix-credential-defaults branch from 19ffebe to c942d62 Compare June 29, 2026 13:58
@tfelix tfelix merged commit fcce9a8 into main Jun 30, 2026
13 checks passed
@tfelix tfelix deleted the feature/fix-credential-defaults branch June 30, 2026 08:28
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