namecoin / nmcontrol

Namecoin Control. This repo is deprecated in favor of https://github.com/namecoin/ncdns
136 stars 38 forks source link

Win output log #93

Closed phelixbtc closed 9 years ago

phelixbtc commented 9 years ago

Allow to view program print output in log file. This should be removed once proper logging is implemented. It is important now for debugging / error reporting, though.

JeremyRand commented 9 years ago

This looks really messy/hacky. Is it really worth doing this when we'll have proper logging in the future anyway? Also, I'm somewhat concerned about writing private data to a file (e.g. websites visited) without the user's knowledge.

phelixbtc commented 9 years ago

This came from the hyperion experiments where debugging was not possible otherwise. The output is overwritten with every launch and should not include website visits (unless debug mode is activated).

Let me ponder on the logging... :)

phelixbtc commented 9 years ago

Waiting for https://github.com/namecoin/nmcontrol/pull/96

phelixbtc commented 9 years ago

As of https://github.com/namecoin/nmcontrol/pull/96 this should not be necessary any more, at least not like this. Closing.