Skip to content

fix: A2A RPC URL got overriden by host and port param of adk api server #2520

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 13, 2025
Merged

fix: A2A RPC URL got overriden by host and port param of adk api server #2520

merged 1 commit into from
Aug 13, 2025

Conversation

copybara-service[bot]
Copy link

fix: A2A RPC URL got overriden by host and port param of adk api server

@copybara-service copybara-service bot added the google-contributor [Bot] This PR is created by Google label Aug 13, 2025
@copybara-service copybara-service bot force-pushed the copybara/794687030 branch 3 times, most recently from 1f82ac2 to 2b519a3 Compare August 13, 2025 20:17
@copybara-service copybara-service bot merged commit 52284b1 into main Aug 13, 2025
@copybara-service copybara-service bot deleted the copybara/794687030 branch August 13, 2025 20:20
@racinmat
Copy link

Hi, could anyone please explain how to expose agents defined in ADK over A2A that I want to deploy to various places?
I need to have the url as a parameter, because if I deploy ADK agent somewhere, and expose it over A2A, then the URL needs to correspond to where it's deployed which can't be hardcoded in a json file, because obviously, the same source code will be deployed to different environments accessible under different URLs.

@racinmat racinmat mentioned this pull request Aug 15, 2025
Open
@racinmat
Copy link

I am fixing this in #2556 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
google-contributor [Bot] This PR is created by Google
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@racinmat @seanzhougoogle