~rjarry/aerc#127: 
review logging and add where appropriate

Now that we have trace and levels and stuff we should go through the codebase and add trace where users are likely to run into trouble.

E.g. it seems send (imap send?) doesn't have any logging which made it hard to help out on IRC.

Status
RESOLVED IMPLEMENTED
Submitter
~ferdinandyb
Assigned to
No-one
Submitted
1 year, 2 months ago
Updated
7 months ago
Labels
feature

~ferdinandyb 1 year, 1 month ago

Some questions/suggestions:

  • add trace to all command Execute calls (with flags?)
  • add trace(?) to all instances of PushStatus?
  • debug for all configuration options of send? i.e. debug where account specific configurations are used?

~ferdinandyb 1 year, 15 days ago

I sent a patch for adding debug to send. Do we see anything else that should warrant keeping the ticket open?

Perhaphs the postmessage, processmessage logs could be improved to provide to provide a bit more info?

~rjarry 1 year, 9 days ago

Bence Ferdinandy referenced this ticket in commit 05c5dcd.

~poldi1405 1 year, 2 days ago

Bence Ferdinandy referenced this ticket in commit 05c5dcd.

~rjarry REPORTED IMPLEMENTED 7 months ago

~ferdinandyb IMPLEMENTED CLOSED 7 months ago

Robin also had a large logging patch, so closing for now.

~ferdinandyb CLOSED REPORTED 7 months ago

~ferdinandyb REPORTED IMPLEMENTED 7 months ago

Register here or Log in to comment, or comment via email.