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.

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

fix(pdk): fix log serialize upstream_status entry nil bug in subrequest case #12953

Merged
merged 9 commits into from
Jun 5, 2024
Merged

fix(pdk): fix log serialize upstream_status entry nil bug in subrequest case #12953

merged 9 commits into from
Jun 5, 2024

Conversation

oowl
Copy link
Member

@oowl oowl commented Apr 28, 2024 •
edited
Loading

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
tzssangglass
tzssangglass reviewed Jun 4, 2024
View reviewed changes
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:

locao pushed a commit that referenced this pull request Jun 21, 2024
@oowl @bungle
fix(pdk): fix log serialize upstream_status entry nil bug in subrequ…
a38d6ae 
…est case (#12953)

FTI-5844

(cherry picked from commit 33765ab)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liverpool8056 liverpool8056 liverpool8056 approved these changes

@catbro666 catbro666 catbro666 approved these changes

@tzssangglass tzssangglass tzssangglass approved these changes

@flrgh flrgh flrgh approved these changes

@windmgc windmgc windmgc approved these changes

Assignees

@oowl oowl

Labels
cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee core/logs size/M
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@oowl @team-gateway-bot @windmgc @flrgh @tzssangglass @catbro666 @liverpool8056