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

Switch logstream command to use Bearer token when basic auth is not a… #3342

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

khkh-ms
Copy link
Contributor

@khkh-ms khkh-ms commented Apr 25, 2023

…llowed.

Changing the log stream command to use bearer token when basic auth is not enabled/allowed.

Issue describing the changes in this PR

resolves #issue_for_this_pr

Pull request checklist

  • My changes do not require documentation changes
    • Otherwise: Documentation issue linked to PR
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • I have added all required tests (Unit tests, E2E tests)

@khkh-ms khkh-ms requested a review from FinVamp1 April 26, 2023 13:07
@khkh-ms
Copy link
Contributor Author

khkh-ms commented Apr 27, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@khkh-ms khkh-ms merged commit 8878ab9 into v4.x Apr 28, 2023
@khkh-ms khkh-ms deleted the khkh/basic-auth-fix branch April 28, 2023 08:13
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