Skip to content

Conversation

@Sara23
Copy link

@Sara23 Sara23 commented Jun 2, 2021

… email.

Q A
Is bugfix? yes
New feature? yes
Breaks BC? no
Fixed issues Check if the user exist before send email.

@maxxer
Copy link
Collaborator

maxxer commented Jun 3, 2021

Why this change? Can't you trap RuntimeException and handle the error in the application, can you?

@TonisOrmisson
Copy link
Contributor

I do not like the mechanics of being able to check if an user exists or not. I think the default "blind" way is good and should not be changed unless any serious arguments are presented.

@maxxer
Copy link
Collaborator

maxxer commented Jul 18, 2022

Agree with @TonisOrmisson. If you want this change to be merged, it should be controlled by a configuration parameter, but the default behavior must be the current one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants