nicolasff / webdis

A Redis HTTP interface with JSON output
https://webd.is
BSD 2-Clause "Simplified" License
2.83k stars 304 forks source link

Simple Logging Class #2

Closed mrb closed 13 years ago

mrb commented 13 years ago

Hey There:

Your project is coming along nicely! Thanks for working on this, it's something that's been needed for a while. I had some free time and I'm trying to get my "C" back after a long time, so I put together this simple logging class that is only partially formed and dropped it into the project. Maybe it will be of use to you or maybe not, but I thought you'd like to check it out.

Mike

nicolasff commented 13 years ago

Hello Michael,

Thanks a lot for this first contribution, I'd say it's a pretty good start! I'll merge this tomorrow and work a little more on it, a few things need to be changed before it's pushed:

Please feel free to suggest anything, not just items that are on the todo/ideas list. It's a young project and there is certainly a lot of things to do; I'm mostly adding what I think is interesting, and listening to possible users. The code base is small enough that a lot of features can be added or changed quickly.

You have already included a log level, and if you feel that some items could go to a different level because it's too verbose, feel free to just open a ticket. Same goes for log files, maybe there could be a different log file for HTTP requests? All suggestions welcome :-)

Cheers, Nicolas

mrb commented 13 years ago

Nicolas,

Thanks! I'll implement a couple of your suggestions as I can and push again. Do you think you want to specify the server verbosity in the config file? I can include that as well.

In terms of what to log where, I'm totally open.

Mike