sandbox should be available to node servers
Closed this issue · 3 comments
The NodeServer creates a 'sandbox' which is a directory in the config dir for that specific SimpleNodeServer instance. This should also be available to the SimpleNodeServer to store logs.
I have implemented the same way @Einstein42 did for isyver for #8.
This is functional and tested.
DEBUG [03-24-2016 20:12:47] polyglot.nodeserver_manager: ihcam STDIN: {"sandbox": {"sandbox": "/home/pi/development/Polyglot/config/camera"}}
Accessible by: self.poly.sandbox from SimpleNodeServer(NodeServer)
If nobody has an issue with it, then I will commit.
Nice. Go ahead. I hadn't gotten around to it yet, I've been busy this week.
Thanks for tackling that.
On Thu, Mar 24, 2016 at 10:42 PM, jimboca notifications@github.com wrote:
The NodeServer creates a 'sandbox' which is a directory in the config dir
for that specific SimpleNodeServer instance. This should also be available
to the SimpleNodeServer to store logs.I have implemented the same way @Einstein42
https://github.com/Einstein42 did for isyver for #8
#8.This is functional and tested.
DEBUG [03-24-2016 20:12:47] polyglot.nodeserver_manager: ihcam STDIN: {"sandbox": {"sandbox": "/home/pi/development/Polyglot/config/camera"}}
Accessible by: self.poly.sandbox from SimpleNodeServer(NodeServer)
I nobody has an issue with it, then I will commit.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#21
No worries, after seeing where sandbox was defined, then seeing how you implemented isyver, it made it easy. I've tested it a bunch so I will commit.
Changes committed to development branch and working.