Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: deepcopy doesn't copy the metatable #6623

Merged
merged 1 commit into from
Mar 16, 2022
Merged

fix: deepcopy doesn't copy the metatable #6623

merged 1 commit into from
Mar 16, 2022

Conversation

spacewander
Copy link
Member

@spacewander spacewander commented Mar 16, 2022 •
edited
Loading

Signed-off-by: spacewander spacewanderlzx@gmail.com

Description

Fixes # (issue)

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@spacewander
fix: deepcopy doesn't copy the metatable
6b15c02 
Signed-off-by: spacewander <spacewanderlzx@gmail.com>
@spacewander spacewander marked this pull request as ready for review March 16, 2022 08:51
@spacewander spacewander merged commit 91b57e6 into apache:master Mar 16, 2022
@spacewander spacewander deleted the ptd branch March 16, 2022 09:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@tokers tokers tokers approved these changes

@leslie-tsang leslie-tsang leslie-tsang approved these changes

Assignees
No one assigned
Labels
need backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@spacewander @tokers @leslie-tsang