alchemist
alchemist copied to clipboard
mgattozzi
[NOMERGE] Add Alchemist Server
This was a lot of work and theres still things to be done, however it should be good for now.
Rocket is the backend for the web server with serde for json, it takes
requests at "http:/// for the index, takes a GET
- /version["/", "/client", "server"] returns the current
versions of the client and server on GitHub respectivly, or both if not
specified.
- /packages Takes POST request with the package[s], distro name,
the name of the client and version of the client.
This is a pretty big change to how Alchemist will work and is a on going progress.
Fixes: #77 On the block to be fixed are: #75 #26 #79
Until the issues mentioned above are fixed this shouldn't be considered for merging/
I was just rebasing the branch to be up to date with master, no actual changes
I pushed some new changes, I want to get #79 working before merging this into master... however all code not related to that should be good for review.
Looks like travis borked its self again, I'll push a commit to fix that when I get the chance. Also going to note that #79 is coming along pretty good, hopefully should be done in a few days.
I don't want to merge this in until I have #79 done as part of this merge, I feel like that is a important part of the server's core functionality.
Ahh I see. Normally if someone puts in a PR and notes why it shouldn't be merged yet then nothing will happen until it's done. That's why I had removed it but good to know you're waiting till you close #79 with it!
Yep, it's being worked on, but if it takes too long and everything else is fixed feel free to merge it. It shouldn't take that long though.
There's no rush your stuff is independent from the other code so this is fine!
@SirDoctors I was wrong with the code I gave you for the filter. You needed to do a filter/map/collect and I forgot that was what was needed. I've updated the code above for you.
@SirDoctors just letting you know I saw this and will get to reviewing when I can!