search

alfetahe / process-hub

Distributed processes manager and global process registry
GNU General Public License v3.0
62 stars 2 forks source link

Fix `stop_child` when using string IDs for the children #1

Closed peaceful-james closed 2 months ago

peaceful-james commented 2 months ago

Invalid call to Keyword.get when using string IDs.

The child IDs have to be either atom or string.

For dynamically created processes, string is the sane option, because String.to_atom is notoriously dangerous.

I had to refactor the tests a bit to be able to write a test for this.

alfetahe commented 2 months ago

Looks good, thank you for the contribution!