EmbarkStudios / crash-handling

Collection of crates to deal with crashes
Apache License 2.0
138 stars 13 forks source link

Unify Unix domain socket client/server implementations #19

Closed Jake-Shadle closed 2 years ago

Jake-Shadle commented 2 years ago

Now with #17 we have 3 different client/server implementations in minidumper for each platform, but now that I've proved that Unix domain sockets can work on all of the (current) target platforms, it makes sense to consolidate them down to a single client and server implementation, with inline platform specifics instead of a bunch of duplicated code. The only big difference between them is the handling of the platform specific crash context, and in the case of linux, the use of abstract names versus filepaths, and the user of ancillary data for retrieving the client PID.

Jake-Shadle commented 2 years ago

Oops, fixed in #23