05nelsonm / kmp-tor

Kotlin Multiplatform Library for embedding Tor into your applications
Apache License 2.0
33 stars 5 forks source link

Remove `torrcFile` and `torrcDefaultsFile` from `TorRuntime.Environment` #429

Closed 05nelsonm closed 1 month ago

05nelsonm commented 1 month ago

Because we're using LOADCONF to load non-cli commands after tor has started up, whatever settings are in the torrc or torrc-defaults file are discarded anyways. In order to incorporate the torrc && torrc-defaults files in that LOADCONF call, there's a lot of parsing that'd need to occur in order to determine what should be overridden... tor already does this if you declare them in the command line arguments. I do not want to do that.....

This will make it so that all settings must be expressed in TorConfig. It also makes is so that if a setting builder is not yet implemented and someone needs it, people are SOL until that happens.

This does not change behavior between 1.x.x and 2.0.0 because LOADCONF was being used there, too.