Skip to content

remove oauth error print for keys - #488

Merged
patelspratik merged 1 commit into
mainfrom
apikeyoautherror
Oct 2, 2026
Merged

patelspratik merged 1 commit into
mainfrom
apikeyoautherror

Conversation

@patelspratik

@patelspratik patelspratik commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

stopped printing the oauth token not found as it is mostly an unnecessary warning rather than a real issue

Comment thread bin/remove-queued-jobs.sh

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removing this because I don't think it adds value

Comment thread pkg/cmd/cmd.go
}
if apiKeyFlag != "" {
os.Setenv(auth.APIKeyEnvVar, apiKeyFlag)
_ = os.Setenv(auth.APIKeyEnvVar, apiKeyFlag)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

cleanliness

Comment thread pkg/store/user.go

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

cleanliness

Comment thread pkg/auth/auth0.go

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

cleanliness

Comment thread pkg/auth/auth.go

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

simple sentinel...I almost removed this altogether, but left it as it seems we still have many auth0 users

@patelspratik
patelspratik marked this pull request as ready for review October 2, 2026 18:10
@patelspratik
patelspratik requested a review from a team as a code owner October 2, 2026 18:10
@patelspratik
patelspratik merged commit f9963d6 into main Oct 2, 2026
9 checks passed
@patelspratik
patelspratik deleted the apikeyoautherror branch October 2, 2026 18:12
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