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

fix connect dial before wrap ssl, revert partial of https://github.co… #11

Closed
wants to merge 1 commit into from

Conversation

bung87
Copy link
Contributor

@bung87 bung87 commented Mar 5, 2024

…m/nim-lang/smtp/commit/7eae0f0d834ffe6517031b4570e9aa86ea7c26bf

fix #12

@ringabout
Copy link
Member

ref #7

cc @mildred

@ThomasTJdev
Copy link

What do we need to get this PR merged? @mildred ?

@ThomasTJdev
Copy link

Any progress here? What would be needed to proceed?

@ringabout
Copy link
Member

ringabout commented Jul 9, 2024

Can we revert 7eae0f0 and #8 fully?

@bung87
Copy link
Contributor Author

bung87 commented Aug 4, 2024

I don't think worth to.

@ThomasTJdev
Copy link

I can remove/redo the code and provide a PR with a detailed description - @ringabout would that be sufficient?

@ringabout
Copy link
Member

I can remove/redo the code and provide a PR with a detailed description -

It would be appreicated, thank you!

@bung87
Copy link
Contributor Author

bung87 commented Aug 16, 2024

close as problem gone by revert

@bung87 bung87 closed this Aug 16, 2024
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.

Ubuntu 20.04 + Nim v2.0 + smtp = No valid socket error code available
3 participants