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

Remove jakarta.mail Dependency #108

Open
rgrashel opened this issue Dec 4, 2023 · 0 comments
Open

Remove jakarta.mail Dependency #108

rgrashel opened this issue Dec 4, 2023 · 0 comments
Assignees
@rgrashel
Labels
dependencies Pull requests that update a dependency file enhancement

Comments

@rgrashel
Copy link
Member

rgrashel commented Dec 4, 2023

I think the jakarta.mail dependency is only used for the EmailTypeConverter for validation purposes. This is completely overkill and unnecessary for email address validation. That logic can be implemented easily using a regex or simple OSS logic that is readily available using other mechanisms. Adding a dependency is not necessary.
@rgrashel rgrashel added enhancement dependencies Pull requests that update a dependency file labels Dec 4, 2023
@rgrashel rgrashel self-assigned this Dec 4, 2023
rgrashel added a commit that referenced this issue Dec 4, 2023
@rgrashel
* Fixed #108
f9f8231
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rgrashel rgrashel

Labels
dependencies Pull requests that update a dependency file enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rgrashel