search

zeek / spicy-analyzers

Growing collection of Spicy-based protocol and file analyzers for Zeek
Other
32 stars 9 forks source link

Fix trans_id indexing with STUN analyzer #75

Closed keithjjones closed 3 years ago

keithjjones commented 3 years ago

I messed up the indexing on the IPV6 logic and it would throw an error in reporter.log. This MR will fix it.

keithjjones commented 3 years ago

The macos_ci test is failing because of homebrew. The tests pass on my macbook.

The other tests fail because of a wireguard testing error. I didn't touch anything wireguard, so those tests failing on this commit doesn't make sense. Everything passes on my macbook:

$ sudo zkg test .
Password:
/Users/[REDACTED]/Source/spicy-analyzers: all tests passed
rsmmr commented 3 years ago

The macos_ci test is failing because of homebrew. The tests pass on my MacBook.

yeah, this is breaking across the board at the moment, there's a Cirrus problem with the base image unfortunately.

The other tests fail because of a wireguard testing error.

This is likewise something that has started to appear elsewhere, needs investigation.

I'll merge.