-
Notifications
You must be signed in to change notification settings - Fork 130
Open
Description
When relaying a mail, the code logs a message about it -- which includes a random UUID:
Lines 176 to 180 in ccb70f5
| logger := log.With(). | |
| Str("from", env.Sender). | |
| Strs("to", env.Recipients). | |
| Str("peer", peerIP). | |
| Str("uuid", generateUUID()). |
What is the rationale here?
- The UUID is not connected to the mail itself (so it would print out a different UUID if it would come through here again for the same mail).
- Thus, there is also nothing to link back from UUID to mail (for tracing, for instance)
- This is the sole location where a UUID is used
- A whole dependency (albeit small) has been added just for this line.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels