~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
REPORTED
Submitter
~ferdinandyb
Assigned to
No-one
Submitted
5 months ago
Updated
a month ago
Labels
feature

~ferdinandyb 4 months 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 3 months 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 3 months ago

Bence Ferdinandy referenced this ticket in commit 05c5dcd.

~poldi1405 3 months ago

Bence Ferdinandy referenced this ticket in commit 05c5dcd.

~rjarry added feature a month ago

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