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.

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

Implemented smtp client certificate validation and introduced 2 new events if smtp server fails to bind to a socket #214

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

vyavorsky
Copy link

@vyavorsky vyavorsky commented Jan 18, 2024

Hi @cosullivan!
I hope this pull request finds you well😅

Thank you for engineering this great lib😄 The flexibility it provides is really awsome, however I ran into a few specific problems that I could not find a reliable workaround for😅 This pull request aims to solve them and extend already impressive feature set of this lib!

I tried to be consistent with your codestyle and implement these features as seamlessly as I could, so everyone who are comfortable with the codebase should feel right at home😃 And of course, all code changes are completely backwards compatible, so no breaking of consumer code!

These features have also been tested with different positive and negative scenarios using stub smtp client, I also tested backwards compatibility so I'm confidend in their stability👍

Additionally I've covered new feature set with tests, added a class with example on how to implement client certificate validation (similarly to existing examples) and I also updated the readme.md to display the new feature😎

I'd really appreciate your review and looking forward to merging these features into main🔥

Have a wonderful day!

P.S. No British words were harmed in the process!😂

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.

2 participants