Skip to content

1784 json serializer decode hashref#1790

Draft
veryrusty wants to merge 2 commits intomainfrom
1784-json-serializer-decode-hashref
Draft

1784 json serializer decode hashref#1790
veryrusty wants to merge 2 commits intomainfrom
1784-json-serializer-decode-hashref

Conversation

@veryrusty
Copy link
Member

Draft patch for JSON serializer _ensure_characters:

  • don't recurse into blessed objects
  • create a new hash in case keys were marked read-only

The test suite will likely fail due to hash randomization with this applied, appropriate use of Test::Differences or similar will sort that out.

Ref::Util is_hashref will tell you if the base variable is a hashref,
even if its blessed. We should be leaving the internals of blessed
objects alone.
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.

1 participant