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

fix(pluginsocket.proto): type for set_upstream #191

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

gszr
Copy link
Member

@gszr gszr commented Mar 12, 2024

Fix the return type for the .Service.SetUpstream external plugin PDK method.

Fixes issue #114.

Sister PR: Kong/kong#12727

@gszr
Copy link
Member Author

gszr commented Mar 12, 2024

Tests failure expected until Kong PR is merged.

@gszr gszr marked this pull request as ready for review March 12, 2024 21:49
gszr added a commit to Kong/kong that referenced this pull request Mar 12, 2024
Fix the return type for the `.Service.SetUpstream` external plugin PDK
method.

Fixes issue Kong/go-pdk#114.

Sister PR: Kong/go-pdk#191
@gszr gszr requested review from StarlightIbuki and locao March 12, 2024 21:55
@gszr gszr marked this pull request as draft March 13, 2024 12:23
@gszr gszr marked this pull request as ready for review March 13, 2024 14:22
gszr added a commit to Kong/kong that referenced this pull request Mar 13, 2024
Fix the return type for the `.Service.SetUpstream` external plugin PDK
method.

Fixes issue Kong/go-pdk#114.

Sister PR: Kong/go-pdk#191
@gszr gszr force-pushed the fix/setupstream-return branch from c5f5348 to 3f8ca70 Compare March 13, 2024 16:25
Fix the return type for the `.Service.SetUpstream` external plugin PDK
method.

Fixes issue #114.

Sister PR: Kong/kong#12727
@gszr gszr force-pushed the fix/setupstream-return branch from 3f8ca70 to cbac0ba Compare March 13, 2024 16:35
@gszr gszr merged commit af1a7d9 into master Mar 13, 2024
3 checks passed
@gszr gszr deleted the fix/setupstream-return branch March 13, 2024 16:40
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