This makes debugging a little easier, and can aid in spotting problems on sites running with Salsa API functionality, without cluttering the screen with connection information.

CommentFileSizeAuthor
dblog.patch1.12 KBgengel

Comments

codewatson’s picture

codewatson commented
Assigned: Unassigned » codewatson

Thanks, will be added with the next commit.

gnassar’s picture

gnassar commented

Why on earth would the watchdog for this be severity 4?

This should be *no* higher than WATCHLOG_INFO or WATCHLOG_DEBUG (6 or 7, respectively).

codewatson’s picture

codewatson commented
Status: Active » Closed (won't fix)

This has been sitting here for a long time, the current code has some log messages for errors. I don't see why you would want to log the response for every query run against salsa, as some page loads could have many queries, unless as you said you are debugging, which most people who use this module outside of development will not, and those that are can create their own debugging?

Closing for now, if you want, reopen!