Support for stable rust

[tglman]

Hi,

To build and run vigil is required nigthly rust, in many case this is fine but in some stable rust is required.
I did my own investigation, and the only reason why nighlty is required is for some compiler features that rocket requires, this should be fixed the next releases of rocket as the master of rocket is already compatible with stable rust: rwf2/Rocket#19 (comment) but this as today is yet not a good option, simply because this is not releases yet.

So i went a bit ahead and made a quick fork (here) that make rocket optional, and then i did a server implementation using actix-web, that as i write it already run in rust stable (you can see the implementation here).

What it's your take on this issue ? do you prefer wait for rocket to be stable ? do you like to have two possible server implementations? or maybe migrate to actix-web is already stable ?

Regards

[valeriansaliou]

Hello @tglman

Thanks a ton for your work on migrating from Rocket to Actix! I was indeed waiting for Rocket to compile on stable, but this takes a bit of time, so I'm willing to fully migrate to Actix.

You may completely remove Rocket in your fork, and submit a PR, I'll greatly appreciate it 👍

Thanks again!

[tglman]

Hi,

Here the PR with rocket that is replaced by actix-web: #72

Regards

[valeriansaliou]

Merged