sim0n00ps / OF-DL

C# console app to download all of the media from Onlyfans accounts with DRM video downloading support
561 stars 54 forks source link

Added tons of extra optional logging #438

Closed thatcoderguy closed 5 days ago

thatcoderguy commented 5 days ago

Hopefully will make debugging much easier. And if this doesn't help, we at least have a way of adding extra debugging in other areas now.

sim0n00ps commented 5 days ago

Awesome thanks mate this should help massively

whimsical-c4lic0 commented 5 days ago

I know this is already merged, but I think the config setting should be changed to set the log level (info, debug, error, etc) rather than a boolean to enable/disable debug logging. The log level config setting should only need to be set in a single location in the code as well rather than passed around between functions.

thatcoderguy commented 5 days ago

I know this is already merged, but I think the config setting should be changed to set the log level (info, debug, error, etc) rather than a boolean to enable/disable debug logging. The log level config setting should only need to be set in a single location in the code as well rather than passed around between functions.

I was thinking about setting the log level in the Log object... I'll make that change shortly :)

whimsical-c4lic0 commented 5 days ago

The log level is currently being set here (https://github.com/sim0n00ps/OF-DL/blob/master/OF%20DL/Program.cs#L35). It should just be a matter of creating a config setting for the log level, and dynamically setting the minimum level on that line instead of having the hard-coded log level.