-
-
Notifications
You must be signed in to change notification settings - Fork 43
Fix some security issues #17
Comments
Yes, this has been pointed out.. I am working on figuring out ways to restrict access outside the browsers, if possible.. This is however, the Docker container, not the host machine.. The container is built as part of this repo so there are no 'real' passwords here.. You can see the 'browser' user being created in the as part of the Dockerfile. |
I understand, but still someone has control over a remote machine, be it a VM or not.. Still not something you'd like - it might be able to break out of the docker container, as it has a network connection (maybe even shared folders) to its host. Thanks for the quick reply :) |
Sure, np, these are tricky issues, and definitely something I want to address, now that this project has gotten quite a lot of traffic. Let me know if you have any specific suggestions, with wine specifically: |
I'll check it out ASAP - looks like a neat project. And yeah, you 're all over tech websites and some trending stuff.. I think you're about to go viral in 2 days |
I think it already has, I can't imagine any more than this! :) |
Well, removing z:/ was causing issues, so undoing.. Browsing root not as significant and can be done from other browsers anyway.. Will look at other security improvements for Wine though |
I have some ideas:
|
I don't think you'll like this kind of stuff on your server my friend;
The text was updated successfully, but these errors were encountered: