/bounty $75

💎 $75 bounty • ZIO

Steps to solve:

1. Start working: Comment /attempt #712 with your implementation plan
2. Submit work: Create a pull request including /claim #712 in the PR body to claim the bounty
3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to zio/zio-schema!

Add a bounty • Share on socials

/attempt #712

Problem

1. The JsonDecoder.decode function in JsonCodec likely parse only the necessary portion of the input string and might ignore trailing characters.
2. This behavior is problematic because it can lead to silent errors where malformed JSON is accepted as valid, potentially causing data inconsistencies or unexpected behavior later in the application.

Potential Solutions

1. Strict Parsing: The most robust solution is to enforce strict JSON parsing. This means the decoder should explicitly check if the entire input string has been consumed after parsing the JSON object. If there are any remaining characters, the decoding should fail with an appropriate error. Here are the key changes that should be made:
   1. In the JsonDecoder object, modify the decode method to check for any remaining non-whitespace characters after parsing is complete. If any are found, it should return an error.
   2. In the string parsing logic (likely in the Lexer object), add a check to ensure there are no extra quotes after the closing quote of a string.
   3. In the object parsing logic, add a check to ensure there are no extra closing braces after the object is closed.

💡 @ImBIOS submitted a pull request that claims the bounty. You can visit your bounty board to reward.

/attempt #712
The inferSchema method is not correctly handling optional fields in case classes. This needs to be addressed to ensure that optional fields are properly recognized and handled.

Update the method to correctly identify and handle optional fields. This involves checking if a field’s type is an Option and adjusting the schema accordingly.

Objective: Ensure optional fields are recognized.
Approach: Use reflection to check if a field’s type is an Option. Update the schema to reflect this.

@Karfue: Reminder that in 7 days the bounty will become up for grabs, so please submit a pull request before then 🙏