amperity / dialog

Simple just-enough logging library for Clojure
MIT License
39 stars 2 forks source link

Dont shell out to get default hostname #8

Closed greglook closed 2 years ago

greglook commented 2 years ago

This is causing a super gnarly issue in our Spark code, due to the following chain of events:

Shelling out in a logging library is probably undesirable even without these class loading shenanigans, so to address this I'm just going to remove the shell call and use the Java-native way of finding the current hostname. This includes a mutator method if clients really want to change the default value after the fact.