Reset bridges if enabling bridges with empty list

Closes #8 (closed)

requested review from @akwizgran

Looks good. Have you tested whether Tor accepts SETCONF UseBridges 1 when no bridge lines are configured? (If it does then I assume it will also accept RESETCONF Bridge when UseBridges is enabled, but maybe worth checking that too.)

Good idea to actually test this :D

Error reply: Unacceptable option value: If you set UseBridges, you must specify at least one bridge.

So that means calling enable bridges with an empty list throws an illegal argument exception or we treat it as disableBridges?

I vote for IllegalArgumentException - faceguns not footguns. ;-)

Changed this MR accordingly.

added 3 commits

• 61604738...3dce2d22 - 2 commits from branch master
• 27fced26 - Throw IllegalArgumentException if bridges are empty

Compare with previous version

merged

mentioned in commit db00ad28