Feature request: validate() to return validated payload

[dracozombie19]

Use case

The Validation feature in Powertools uses fastjsonschema.validate() to perform the validation. This function in fastjsonschema returns the validated payload. While usually this is exactly the same as the input payload, fastjsonschema also populates default values in missing fields. For example, with this schema:

{
  "properties": {
   "required_id": {
    "type": "integer"
   },
   "test_default": {
    "type": "string",
    "default": "def"
   }
  },
  "required": [
   "required_id"
  ],
  "type": "object"
}

And this payload:

{
  "required_id": 1234
}

The validate() function will return:

{
  "required_id": 1234,
  "test_default": "def"
}

Solution/User Experience

I'm wondering if we can have the Powertools validate() function return the results from fastjsonschema.validate().

Alternative solutions

No response

Acknowledgment

• This feature request meets Powertools for AWS Lambda (Python) Tenets
• Should this be considered in other Powertools for AWS Lambda languages? i.e. Java, TypeScript, and .NET

[boring-cyborg]

Thanks for opening your first issue here! We'll come back to you as soon as we can.
In the meantime, check out the #python channel on our Powertools for AWS Lambda Discord: Invite link

[heitorlessa]

Nice!! Didn’t know it was a thing (default values) — feel free to make a pull request and a simple test to confirm the returned value, and we can prioritize it for this Thursday’s release (otherwise when we have the bandwidth to do so). Thanks a lot for flagging it

…

On Mon, 22 Jul 2024 at 17:48, boring-cyborg[bot] ***@***.***> wrote: Thanks for opening your first issue here! We'll come back to you as soon as we can. In the meantime, check out the #python channel on our Powertools for AWS Lambda Discord: Invite link <https://discord.gg/B8zZKbbyET> — Reply to this email directly, view it on GitHub <#4797 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZPQBAEOGUAGIYJPFFGA7DZNUSVZAVCNFSM6AAAAABLITB546VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBTGI4DCNBQGU> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.*** .com>

[dracozombie19]

I did some work on this last week but ran into trouble getting the tests to run in the cloud development environment for some reason. I'll try to revisit it this week.

[dracozombie19]

See: #4839

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

This issue is now closed. Please be mindful that future comments are hard for our team to see.

If you need more assistance, please either tag a team member or open a new issue that references this one.

If you wish to keep having a conversation with other community members under this issue feel free to do so.