dnsproxy icon indicating copy to clipboard operation
dnsproxy copied to clipboard

Get rid of `github.com/jessevdk/go-flags` dependency

Open EugeneOne1 opened this issue 4 years ago • 1 comments

Currently, we're using github.com/jessevdk/go-flags module to handle configurable options in CLI tool but it provides some notable restrictions and design issues. For example, in #170 which adds the .yaml configuration file support we've came across the inability to properly combine the parsed options' values with values from the file.

Moreover, the functionality provided by go-flags doesn't actually justify abandoning of flag from standard library. Since we strive to keep unnecessary dependencies to a minimum, the one should be removed.

EugeneOne1 avatar Oct 21 '21 12:10 EugeneOne1

@Potterli20, see https://github.com/AdguardTeam/dnsproxy/blob/master/config.yaml.dist for an example.

Also, please keep questions not relevant to the subject to a minimum. Thanks.

ainar-g avatar Oct 27 '21 09:10 ainar-g