Jake Shadle

Results 189 comments of Jake Shadle

I guess one thing to note is that I don't think the file path is being taken in any test so I'm not sure about that one, I'll add a...

Ahh sounds good, I just want to add a test specifically for the file case since that will never currently be hit on CI/my local machine, then we can merge.

I think something like this would be great. The MacOS dump_syms stuff I added was loosely based on the end-to-end testing I did in https://github.com/EmbarkStudios/crash-handling/tree/main/minidumper-test though there I at the...

Yah that has been problematic in the past it's fine to relax it.

Nope, I think it would help a bit on Linux, but Windows doesn't need it, and Mac is already just going directly to [GUID](https://github.com/rust-minidump/rust-minidump/pull/534/files#diff-8896d41e2ef879015f9341e2add2bf3fffe128fd01f89943fff39d7320378a84R549-R586) since that's the type in the...

I've created a PoC at https://github.com/Jake-Shadle/ebpf-test to see how feasible this would be. Note that the PoC supports (almost) exactly our actual use case of client packets w/token proxy server,...

> If I want to change the current packet format (which I have been looking at), right now I would just update the config file, with what you're currently proposing,...

Maybe changing to use [AF_XDP](https://docs.ebpf.io/linux/concepts/af_xdp) is the easiest option then, the eBPF program would be solely used to redirect the packets we care about to be processed in userspace similarly...

Yes, I believe it would fail to attach if it's not supported, so easy enough to fallback to the normal I/O path in that case.

Can you provide a link to a full cargo.toml/workspace that repros this behavior, I'm missing too much context though I know what needs to be fixed.