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

feat(pdk): add service.set_proxy_address function #13775

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

locao
Copy link
Contributor

@locao locao commented Oct 18, 2024

Summary

Add a new PDK function to set the IP and port to proxy the request, overwriting the result from the load-balancer.

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

KAG-5628

@github-actions github-actions bot added core/pdk cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Oct 18, 2024
@team-gateway-bot team-gateway-bot added the author/community PRs from the open-source community (not Kong Inc) label Oct 18, 2024
@locao locao force-pushed the feat/set_service_proxy_host branch from 11ffba0 to 4b5b3a5 Compare October 18, 2024 15:24
@locao locao marked this pull request as draft October 23, 2024 14:16
@locao
Copy link
Contributor Author

locao commented Oct 23, 2024

I changed this PR back to draft while we make a decision if we want to add this method or change the behavior of the existent one. Please feel free to review and add comments about the new code itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author/community PRs from the open-source community (not Kong Inc) cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee core/pdk size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants