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

JSON parser performs byte serialization before JSON CLType is fully parsed when CLType is after bytes field in JSON structure #274

Closed
cnorburn opened this issue May 20, 2024 · 0 comments · Fixed by #275
Closed

JSON parser performs byte serialization before JSON CLType is fully parsed when CLType is after bytes field in JSON structure #274

cnorburn opened this issue May 20, 2024 · 0 comments · Fixed by #275
Assignees
@meywood
Labels
bug Something isn't working

Comments

@cnorburn
Copy link
Collaborator

deploy-5.json
@meywood meywood changed the title Serialisation bug with multiple deploy args and module_bytes JSON parser performs byte serialization before JSON CLType is fully parsed when CLType is after bytes field in JSON structure May 20, 2024
@meywood meywood added the bug Something isn't working label May 20, 2024
@meywood meywood self-assigned this May 20, 2024
@meywood meywood linked a pull request May 20, 2024 that will close this issue
Issues/274 - Refactor JSON deserialization to use @JsonCreator to ensure that cl_type is parsed prior to setting any value fields in AbstractCLValueWithChildren and CLValueByteArray classes. #275
Merged
cnorburn added a commit that referenced this issue May 20, 2024
@cnorburn
Merge pull request #275 from /issues/274
b848eda 
Issues/274 - Refactor JSON deserialization to use  @JsonCreator to ensure that cl_type is parsed prior to setting any value fields in AbstractCLValueWithChildren and CLValueByteArray classes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@meywood meywood

Labels
bug Something isn't working
Projects
Work In Progress
Status: Done
Milestone
No milestone
2 participants
@cnorburn @meywood