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

integrate Zeek IEC104 parser #379

Open
mmguero opened this issue Nov 4, 2024 · 1 comment
Open

integrate Zeek IEC104 parser #379

mmguero opened this issue Nov 4, 2024 · 1 comment
Labels
enhancement New feature or request ics Relating to ICS (Industrial Control Systems) devices zeek Relating to Malcolm's use of Zeek
Milestone

Comments

@mmguero
Copy link
Collaborator

mmguero commented Nov 4, 2024

@mmguero cloned issue idaholab/Malcolm#557 on 2024-09-04:

cert-lv/zeek-iec104 takes the initial work done by @georgemakrakis and expands on it. I think this would be a great protocol parser to integrate into Malcolm.

see:

@mmguero mmguero added enhancement New feature or request ics Relating to ICS (Industrial Control Systems) devices zeek Relating to Malcolm's use of Zeek labels Nov 4, 2024
@mmguero
Copy link
Collaborator Author

mmguero commented Nov 4, 2024

@mmguero commented on 2024-09-11:

There are a few issues I'd like to resolve in the plugin before getting this in:

  • can we make it signature based rather than port based?
  • I don't like the way it overrides JSON vs. TSV, is there a reason we can't just let Zeek handle it globally like all the other plugins?
  • this isn't as huge a deal, but the tests directory is kind of weird

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ics Relating to ICS (Industrial Control Systems) devices zeek Relating to Malcolm's use of Zeek
Projects
Status: No status
Development

No branches or pull requests

1 participant