chore(deps): remove email-validator; use Str over EmailStr in SES model

[rubenfonseca]

Issue number: #1607

Summary

Changes

Removes email-validator from the extra group dependencies.

Please provide a summary of what's being changed

User experience

This will also reduce the total Lambda Layer and SAR app size.

Please share what the user experience looks like before and after this change

Checklist

If your change doesn't seem to apply, please leave them unchecked.

• Meet tenets criteria
• I have performed a self-review of this change
• Changes have been tested
• Changes are documented
• PR title follows conventional commit semantics

Is this a breaking change?

RFC issue number:

Checklist:

• Migration process documented
• Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

[codecov-commenter]

Codecov Report

Base: 99.38% // Head: 99.40% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (6507bee) compared to base (4b76eca).
Patch coverage: 91.46% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##               v2    #1608      +/-   ##
==========================================
+ Coverage   99.38%   99.40%   +0.01%     
==========================================
  Files         125      127       +2     
  Lines        5731     5836     +105     
  Branches      359      367       +8     
==========================================
+ Hits         5696     5801     +105     
+ Misses         18       17       -1     
- Partials       17       18       +1     

Impacted Files	Coverage Δ
aws_lambda_powertools/tracing/extensions.py	100.00% <ø> (ø)
aws_lambda_powertools/utilities/batch/__init__.py	100.00% <ø> (ø)
...ws_lambda_powertools/utilities/batch/exceptions.py	100.00% <ø> (ø)
...a_powertools/utilities/parser/envelopes/kinesis.py	100.00% <ø> (ø)
aws_lambda_powertools/shared/cookies.py	58.69% <58.69%> (ø)
...bda_powertools/utilities/data_classes/alb_event.py	90.47% <66.66%> (ø)
...ambda_powertools/utilities/parameters/appconfig.py	94.28% <86.66%> (ø)
aws_lambda_powertools/utilities/batch/base.py	97.79% <87.50%> (-1.42%)	⬇️
aws_lambda_powertools/event_handler/api_gateway.py	100.00% <100.00%> (ø)
aws_lambda_powertools/event_handler/appsync.py	100.00% <100.00%> (ø)
... and 41 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ran-isenberg]

Hey @rubenfonseca @heitorlessa , that's a breaking change. users are expecting here proper email address validation.
I think a better solution would be:

1. During import try to import EmailStr, if there's an except redefine the type as str
2. add docs and specify that for Ses model, you should install the extra dependency

WDYT?

[rubenfonseca]

Hi @ran-isenberg , we hear you and appreciate your feedback. We’re including this change in V2 to not impact customers in any shape or form.

To add to my comment earlier, as helpful as e-mail validator may be for an untrusted payload, this is not the case for SES. It would, however, for something like “body” in API Gateway.

I’ll update the issue linked with this and related Pydantic overhead that are making us more cautious on double validation.

[heitorlessa]

Reviewing it now.

Before merging, we need to make sure @huonw @mgochoa are credited. Let's try using the recently discovered manual co-authors feature in GitHub. We should do it in the Release notes regardless