Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Extract common utilities #2

Open
photm5 opened this issue Jun 30, 2015 · 2 comments
Open

Extract common utilities #2

photm5 opened this issue Jun 30, 2015 · 2 comments
Labels
todo

Comments

@photm5
Copy link
Owner

photm5 commented Jun 30, 2015

This issue depends on #1.

  • Server and supervisor both need the same message parsing and command framework
  • Server, visualizer, and sniffer all need a TCP accept loop
@photm5
Copy link
Owner Author

photm5 commented Jul 4, 2015

I decided on combining all the seperate cabal projects into one, but leaving the executables seperated.

@photm5 photm5 added todo and removed decision labels Jul 4, 2015
photm5 added a commit that referenced this issue Jul 4, 2015
@photm5
Merge branch 'cabal-merge'
16d725e 
Fixes #5
This will be useful for #2.
@photm5
Copy link
Owner Author

photm5 commented Jul 4, 2015

Commit e85b598 merges in some nice things, but there are more things to extract. From the top of my head:

  • Server and supervisor should have a way to wait for messages with a certain ID
  • Sniffer and supervisor should reconnect when the connection is lost (See Add some failure safety #6)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
todo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@photm5