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

tokio-core is deprecated. #29

Closed
quadrupleslap opened this issue Aug 4, 2018 · 9 comments
Closed

tokio-core is deprecated. #29

quadrupleslap opened this issue Aug 4, 2018 · 9 comments

Comments

@quadrupleslap
Copy link

No description provided.

@bbigras
Copy link

bbigras commented Sep 11, 2018

Any progress on this?

@masonk
Copy link

masonk commented Sep 27, 2018

I think that conversion from tokio_core to tokio is blocking the use of ldap3 with any other tokio system, such as actix-web (and many others).

@inejge
Copy link
Owner

inejge commented Sep 27, 2018

Just a short note (as I otherwise don't have the time to work on this library): moving away from tokio_core to tokio would have to account for the use of the former in tokio_proto, also deprecated but unfortunately the foundation of ldap3. I don't see a straightforward way to excise tokio_proto without a massive rewrite of this crate's internals.

@masonk
Copy link

masonk commented Sep 27, 2018

Yes, I was realizing that last night when I was trying to do the migration myself.

@nevi-me
Copy link

nevi-me commented Mar 23, 2019

Hi @inejge , I had a go at trying to port to tokio today, and have made some progress. It looks like the blocker is tokio_proto more than anything, for the multiplex tcp connections.

I'm due to release a program that will be deployed at clients in the coming days, so I don't have a choice but to "make it work".
I'm going to try something from the tower ecosystem and see if it works. I'll provide feedback in a few days

@inejge
Copy link
Owner

inejge commented Mar 23, 2019

Right, the tokio_proto multiplexer is the tricky part, and having it available is what made initial development comparatively straightforward. For your immediate problem, I'd recommend starting a dedicated thread for LDAP connection(s), with its own tokio_core event loop, and communicating with the rest of the system via oneshot/mpsc channels. I believe these can work across tokio flavors.

@nevi-me
Copy link

nevi-me commented Mar 23, 2019

I found red75prime/tokio-proto@cd1af82, which while not ideal, still allowed me to get the crate to build. I just got it to build now actually.

@inejge
Copy link
Owner

inejge commented Apr 28, 2020

With the port to Tokio 0.2, this can now be closed.

@inejge inejge closed this as completed Apr 28, 2020
@nevi-me
Copy link

nevi-me commented Apr 28, 2020

Thanks @inejge

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

No branches or pull requests

5 participants