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(pdk): fix log serialize upstream_status entry nil bug in subrequest case #12953

Merged
merged 9 commits into from
Jun 5, 2024

Conversation

oowl
Copy link
Member

@oowl oowl commented Apr 28, 2024

Summary

PDK: fix log serialize upstream_status entry nil bug in subrequest case

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

Fix FTI-5844

@github-actions github-actions bot added core/logs cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Apr 28, 2024
@pull-request-size pull-request-size bot added size/M and removed size/XS labels May 6, 2024
@oowl oowl force-pushed the fix/pdk-upstream-status-lost branch 5 times, most recently from f1691fb to 56abb1e Compare May 6, 2024 09:16
kong/pdk/log.lua Outdated Show resolved Hide resolved
@oowl oowl force-pushed the fix/pdk-upstream-status-lost branch from 56abb1e to 20a55cf Compare June 5, 2024 03:12
@oowl oowl force-pushed the fix/pdk-upstream-status-lost branch from 20a55cf to 43d7eeb Compare June 5, 2024 03:12
@windmgc windmgc merged commit 33765ab into master Jun 5, 2024
25 checks passed
@windmgc windmgc deleted the fix/pdk-upstream-status-lost branch June 5, 2024 05:29
@team-gateway-bot
Copy link
Collaborator

Successfully created cherry-pick PR for master:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee core/logs size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants