News for hackers. Stuff that matters.
Re:Rust's 2017 Roadmap by steveklabnik on Monday February 06, 2017 @05:51PM (#13585074)
I am about to go into two hours of meetings so this will have to be quick.

The issue is, because this is a trait, you must conform to its signature. Which is here: https://hyper.rs/hyper/v0.10.0/hyper/server/trait.Handler.ht... and specifically https://hyper.rs/hyper/v0.10.0/hyper/server/trait.Handler.ht...

So yeah, my idea won't work, because those types conflict.

> Removing the trait impl was another thing recommended by the Arc guy at work, he didn't know how or why it worked.

I'm surprised it works on first glance, as I was assuming that you were implementing the trait for a reason, not just because. If you don't need the trait, then yeah, it's much easier.

> What's different with Rust?

See my StackExchange link elsewhere in the thread for a summary.

I'll maybe poke at this after my meetings, or maybe tomorrow. We'll see how amped I am for coding after all that ;)

Re:Rust's 2017 Roadmap by ainar-g on Monday February 06, 2017 @05:56PM (#13585117)
Okay, thank you again for your time!
Re:Rust's 2017 Roadmap by steveklabnik on Tuesday February 07, 2017 @12:55PM (#13591132)
Ah! I understand now.

Handlers aren't meant to be nested like this. That is, it's always going to break this way, because the handler needs to live as long as the request and response, so creating a sub-handler is not ever going to live long enough.

So yes, the solution is to not implement Handler for your UserIndexHandler; then you can just call it.

I bet there's something larger you could do as well, but I don't have enough experience with this interface to give good advice. Hyper's undergoing a huge docs drive for 0.11; I'm sure that'll help.

Re:Rust's 2017 Roadmap by ainar-g on Tuesday February 07, 2017 @05:25PM (#13593507)
Yeah, I figured that out as well, but thank you for finding the time to analyse the code and write an answer. I appreciate it and may be on my fourth attempt at actually learning Rust :)
Re:Rust's 2017 Roadmap by steveklabnik on Tuesday February 07, 2017 @05:43PM (#13593668)
No problem. :)