Skip to content

fix(tools): support OpenAPI basic auth#5493

Open
aashnakunk wants to merge 2 commits intogoogle:mainfrom
aashnakunk:fix/openapi-basic-auth
Open

fix(tools): support OpenAPI basic auth#5493
aashnakunk wants to merge 2 commits intogoogle:mainfrom
aashnakunk:fix/openapi-basic-auth

Conversation

@aashnakunk
Copy link
Copy Markdown

@aashnakunk aashnakunk commented Apr 26, 2026

Fixes #5493

Summary

  • Add HTTP Basic auth header generation for OpenAPI tools
  • Add unit coverage for Basic auth credentials

Testing

  • .venv/bin/pytest tests/unittests/tools/openapi_tool/auth/test_auth_helper.py

Result:

  • 36 passed, 1 warning in 9.32s

@google-cla
Copy link
Copy Markdown

google-cla Bot commented Apr 26, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@adk-bot adk-bot added the tools [Component] This issue is related to tools label Apr 26, 2026
@adk-bot
Copy link
Copy Markdown
Collaborator

adk-bot commented Apr 26, 2026

Response from ADK Triaging Agent

Hello @aashnakunk, thank you for creating this PR!

Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). You can visit https://cla.developers.google.com/ to see your current agreements or to sign a new one.

Also, this PR is a bug fix. Could you please associate the GitHub issue with this PR? If there is no existing issue, could you please create one?

This information will help reviewers to review your PR more efficiently. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tools [Component] This issue is related to tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants