mongodb-labs / edda

A log visualizer for MongoDB - This Repository is NOT a supported MongoDB product
232 stars 28 forks source link

Take args that mongo tools take #5

Closed kchodorow closed 12 years ago

kchodorow commented 12 years ago

E.g.,

options: --help produce help message -v [ --verbose ] be more verbose (include multiple times for more verbosity e.g. -vvvvv) --version print the program's version and exit -h [ --host ] arg mongo host to connect to ( /s1,s2 for sets) --port arg server port. Can also use --host hostname:port

// less important: --ipv6 enable IPv6 support (disabled by default) -u [ --username ] arg username -p [ --password ] arg password -d [ --db ] arg database to use -c [ --collection ] arg collection to use (some commands)

samantharitter commented 12 years ago

When running the --username and --password options, I was thinking that it would be good to collect the more sensitive information using getpass instead of having it echoed on the command line: http://docs.python.org/library/getpass.html

kchodorow commented 12 years ago

Good call.

samantharitter commented 12 years ago

Should auth information be provided in the command line at all? Would it be better to prompt for this information if, when the program tried to connect to the given db, it required authentication?

kchodorow commented 12 years ago

I'd much rather have you work on https://github.com/kchodorow/logl/issues/6 than get all the extra command line opts working.

samantharitter commented 12 years ago

As of now, the program handles the following arguments correctly: --help --port --host (and --host hostname:port) --verbose, -v(v)(v) --version --db, -d --collection, -c

Are these features enough for now?

kchodorow commented 12 years ago

Yeah, definitely.

samantharitter commented 12 years ago

Can I temporarily close this issue? Or is it better to leave it open?

kchodorow commented 12 years ago

Close it, I'll make a different issue for extended options.

gekitsuu commented 12 years ago

It would be great if this had a link to the other issue that was opened. I search for authentication and found this as the only ticket. Would be handy to have a link to https://github.com/10gen-labs/edda/issues/16