Fix: optional param's default handling

[lohanidamodar]

• When an optional parameter is sent in the request with explicit null value, set it to default if provided

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

• 🔍 Trigger review

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-optional-default-param

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[abnegate]

What if we want to actually pass null to an optional param?

[lohanidamodar]

what should we do? should we check if validator is nullable ? what do we do? if null value is supported, shouldn't default be null? do we have a case where we support null but also have something else as default value?