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.

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

tshark errors silently ignored #64

Open
eprochasson opened this issue Aug 16, 2024 · 3 comments
Open

tshark errors silently ignored #64

eprochasson opened this issue Aug 16, 2024 · 3 comments

Comments

@eprochasson
Copy link

Hi,

it looks like tshark errors are silently ignored, which leads to other problems down the line. Typically, I got a "Found no signals, are you sure supports monitor mode?" error, first because:

  • I didn't have the right permission to run tshark, which I fixed
  • then, having run some tests in root, the dumpfile used could not be overwritten by non-root user.

In these cases, instead of getting a clear error message from the subprocess, I just got the fairly unhelpful error from the script.

I would suggest the following:

  • show any error from tshark as a warning. Some errors are not a problem (I do get another one, that doesn't seem to hinder the proper flow), so we may not want to abort the process on error.
  • use proper temporary files / directory, to avoid the type of collision I encountered
  • maybe to use logging, to have a simpler way to manage warning / info / debug (=verbose) stuff.

I'm happy to give it a shot and submit a PR, if it's something that is deemed valuable.

Thanks!

@sulrich70
Copy link

I would appreciate it, thats for sure. I have a project where i want to count attendance at a convention, in several buildings. I was hoping to use raspberry pi zeros to do this…

@eprochasson
Copy link
Author

Well the core tool still works. Those are quality of life improvement. I'm more asking @schollz if he's interested in such contribution :)

@schollz
Copy link
Owner

schollz commented Aug 17, 2024

Absolutely! I'll merge a PR and version up no problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants