katzenpost / mailproxy

POP/SMTP to Katzenpost proxy server library
GNU Affero General Public License v3.0
4 stars 5 forks source link

api: It kind of sucks. #23

Open Yawning opened 6 years ago

Yawning commented 6 years ago

mailproxy was written to be a standalone daemon above anything else, and it shows. The API could be made a lot better by exposing more control and information such as:

I see this effort as being an on going task based on feedback from consumers of the API, and it would be nice if people that aren't me tackled this along with #4.

Valodim commented 6 years ago

I can make use of almost any event that happens:

Yawning commented 6 years ago

Ok. I went and added events for:

The one area lacking event reporting is failure to fetch the PKI directly from the authority (though you'll get an event saying that the connection attempt has failed due to the lack of a document/descriptor after a while). There should be events for PKI fetch failures, but the way the code is structured makes this somewhat annoying to implement, so I'm punting on this for now.

david415 commented 6 years ago

I do not get the Send completion event: https://github.com/katzenpost/tools/blob/add_zcash_wallet_proxy.0/loopix_ping/main.go

Also it doesn't shutdown but instead hangs... but maybe that's because I've got a mistake somewhere.