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

"brimcap search" command line tripped up by paths containing space chars #1597

Closed
philrz opened this issue Apr 30, 2021 · 1 comment · Fixed by #1598
Closed

"brimcap search" command line tripped up by paths containing space chars #1597

philrz opened this issue Apr 30, 2021 · 1 comment · Fixed by #1598
Labels
bug Something isn't working

Comments

@philrz
Copy link
Contributor

philrz commented Apr 30, 2021

Repro is with the nightly build https://storage.googleapis.com/brimsec-releases/brim/v0.25.0-prerelease-60788403.0/macos/Brim-0.25.0-prerelease-60788403.0.dmg, hence Brim commit 6078840 which talks to zed commit aa4170e.

If I open the attached hello.pcapng.gz (after uncompressing), then attempt to click the shark fin to extract a flow from it, I'm shown the "Error preparing pcap" pop-up.

image

As shown in the Dev Tools, the problem seems to be a simple case of the pathname not being quoted and hence the space character in Application Support is truncating the path.

Error: Command failed: /Applications/Brim.app/Contents/Resources/app.asar.unpacked/zdeps/brimcap search -dst.ip 54.210.89.192 -dst.port 443 -duration 0.293493s -proto tcp -root /Users/phil/Library/Application Support/Brim/data/brimcap-root -src.ip 192.168.10.120 -src.port 63342 -ts 2020-03-09T17:27:53.399273000Z -w /var/folders/yn/jbkxxkpd4vg142pc3_bd_krc0000gn/T/packets-1583774873.399273.pcap
{"type":"error","error":"stat /Users/phil/Library/Application: no such file or directory"}
@philrz philrz added the bug Something isn't working label Apr 30, 2021
@philrz
Copy link
Contributor Author

philrz commented May 3, 2021

Verified with nightly build https://storage.googleapis.com/brimsec-releases/brim/v0.25.0-prerelease-09efdcd9.0/macos/Brim-0.25.0-prerelease-09efdcd9.0.dmg, hence Brim commit 09efdcd which talks to Zed commit 709c769.

Per the attached video, now I can open flows from my pcap without any problems involving space characters. The symptom of an unrelated issue brimdata/zed#2645 happens to show up here, FWIW.

Verify.mp4

Thanks @mason-fish!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant