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

Update fast-socks5 dependency #36

Merged
merged 3 commits into from
Mar 15, 2021
Merged

Update fast-socks5 dependency #36

merged 3 commits into from
Mar 15, 2021

Conversation

amaury1093
Copy link
Contributor

No description provided.

mre added a commit to lycheeverse/check-if-email-exists that referenced this pull request Feb 8, 2021
This includes updating Tokio to 1.x.
The feature flags of Tokio have been changed and `dns` was merged into `net`.

`fast-socks5` was used in two different versions because of a dependency in async-smtp. This gets addressed in chatmail/async-smtp#36, but in the meantime we can divert to the fix from the fork at https://github.com/amaurymartiny/async-smtp/tree/am-fast-socks.
amaury1093 pushed a commit to reacherhq/check-if-email-exists that referenced this pull request Feb 8, 2021
* Update dependencies

This includes updating Tokio to 1.x.
The feature flags of Tokio have been changed and `dns` was merged into `net`.

`fast-socks5` was used in two different versions because of a dependency in async-smtp. This gets addressed in chatmail/async-smtp#36, but in the meantime we can divert to the fix from the fork at https://github.com/amaurymartiny/async-smtp/tree/am-fast-socks.

* Use SmallRng, which is Send+Sync

Hyper requires shared types to be Send+Sync. The threaded rng is not.
Replacing it with SmallRng does the trick,
which is satisfactory for creating random mail addresses.

* cargo fmt

* Mark tests with tokio_test

* Update hyper

* Update test suite to Tokio 1.x
@amaury1093
Copy link
Contributor Author

Gentle ping on this PR 🙏

@mre
Copy link

mre commented Mar 15, 2021

@dignifiedquire can we merge this? It's currently blocking the homebrew release of lychee. 😕

@dignifiedquire dignifiedquire merged commit fc532c3 into chatmail:master Mar 15, 2021
@amaury1093 amaury1093 deleted the am-fast-socks branch March 15, 2021 11:38
@amaury1093
Copy link
Contributor Author

Thanks @dignifiedquire, any chance we can get a 0.3.5?

@mre
Copy link

mre commented Mar 23, 2021

Yup, a new 0.3.5 release would be mega @dignifiedquire. Sorry for being so pushy, but our downstream crates depend on it for their next releases. 😅

@dignifiedquire
Copy link
Collaborator

finally got around to publishing 0.4.0

@mre
Copy link

mre commented Mar 29, 2021

Thanks!

coinbest90726 added a commit to coinbest90726/check-email that referenced this pull request Apr 26, 2024
* Update dependencies

This includes updating Tokio to 1.x.
The feature flags of Tokio have been changed and `dns` was merged into `net`.

`fast-socks5` was used in two different versions because of a dependency in async-smtp. This gets addressed in chatmail/async-smtp#36, but in the meantime we can divert to the fix from the fork at https://github.com/amaurymartiny/async-smtp/tree/am-fast-socks.

* Use SmallRng, which is Send+Sync

Hyper requires shared types to be Send+Sync. The threaded rng is not.
Replacing it with SmallRng does the trick,
which is satisfactory for creating random mail addresses.

* cargo fmt

* Mark tests with tokio_test

* Update hyper

* Update test suite to Tokio 1.x
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.

4 participants