search

VOLTTRON / volttron

VOLTTRON Distributed Control System Platform
https://volttron.readthedocs.io/
Other
452 stars 217 forks source link

Fix for security issue #3168 #3169

Closed schandrika closed 2 months ago

schandrika commented 2 months ago

Fix and test case for issue #3168

craig8 commented 2 months ago

@schandrika looks like there are a couple of failures. The utils looks like a AMQP timeout. Like the rmq server stopped quicker than we would like.

However, the auth seems to be a genuine error. https://github.com/VOLTTRON/volttron/actions/runs/8665217491/job/23763385735#step:4:2167
Please take a look and run this on your system.

schandrika commented 2 months ago

@craig8 The auth test pass for me locally. The one failing test in auth here, does depend on gevent.sleep(4) so could be a timing issue. Testutils failure is also a final cleanup timeout and the tests themselves pass