Patrick Schleizer: > Hello anonym! > > anonym: >> Feel free to send a PR with your other >> changes applied to tor-controlport-filter in Tails Git! >> Otherwise >> I'll do it myself later this week. > > Joy rebased Whonix's changes on top of your new version. > > base: > https://git-tails.immerda.ch/tails/plain/config/chroot_local-includes/usr/local/lib/tor-controlport-filter > > fork: > https://github.com/joysn/control-port-filter-python/blob/master/usr/lib/tor-controlport-filter > > The diff looks simple, I guess.
If you see my email from earlier today, I already did this: https://mailman.boum.org/pipermail/tails-dev/2017-January/011190.html > Please ignore: > > - config parser changes I did! However, in your repo I still see that commit bed6399b contains the merge_yml() code. You are gonna do that externally, right? However, the commit talks about "add /etc/tor-controlport-filter.d configuration support", so perhaps it was a mistake (i.e. you wanted to add a `--filter-dir` option, but picked the wrong commit)? > Please take: > > - #!/usr/bin/python3 -u (makes eventual python exceptions and up in journal) > - Use yml.safe_load and Python exceptions in journalctl > - add --listen_interface option These were the commits I imported. > One minor bug, I think, first line is supposed to be > > #!/usr/bin/python3 -u > (not -v) That error was not present in the code I looked at. Strange. Cheers! _______________________________________________ Tails-dev mailing list Tails-dev@boum.org https://mailman.boum.org/listinfo/tails-dev To unsubscribe from this list, send an empty email to tails-dev-unsubscr...@boum.org.