search

dedis / Dissent

Provably Anonymous Overlay
http://dedis.cs.yale.edu/2010/anon/
322 stars 46 forks source link

Test cases and fix to SessionWebService #4

Closed henrycg closed 12 years ago

henrycg commented 12 years ago

Hi David,

Do you want to take a look at this code and see what you think? I implemented the session hack for now and it seems to work just fine.

Henry

davidiw commented 12 years ago

Looks good, two minor gripes

henrycg commented 12 years ago

Ok, I removed conf/web.

davidiw commented 12 years ago

Pulled, could you disable the std out printed in the tests (about where the web server is). I'd eventually like to see a shutdown dissent webservice.

henrycg commented 12 years ago

On 12/09/2011 07:51 PM, davidiw wrote:

Pulled, could you disable the std out printed in the tests (about where the web server is). I'd eventually like to see a shutdown dissent webservice.

Reply to this email directly or view it on GitHub: https://github.com/davidiw/Dissent/pull/4#issuecomment-3088291

I use the stdin/stdout stuff to communicate with the server because it makes easier to test a clean shutdown with valgrind (by sending "quit" on stdin). Catching ctrl-C with Qt is not portable and also requires a lot of signal handling code.