Use the official docker api library

docker / dockercraft

Docker + Minecraft = Dockercraft

Apache License 2.0

7.19k stars 389 forks source link

Use the official docker api library #84

Closed dave-tucker closed 7 years ago

dave-tucker commented 7 years ago

This PR uses docker/client instead of the un-official library. This should be always up-to-date 🎉

Tagged as WIP as I found a bug where all of your containers get nuked 😮 I believed this could have been API related but so far it appears to be some derpiness in the LUA code. I suspect I'm to blame, so tracking it down... and hope to fix before merging this PR

dave-tucker commented 7 years ago

Investigation didn't lead anywhere but I don't think this PR make it any worse. Merging and will open an issue to investigate container derpiness later