Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

Multi-User Mails Redundancy #6

Open
InternalLoss opened this issue Nov 26, 2017 · 2 comments
Open

Multi-User Mails Redundancy #6

InternalLoss opened this issue Nov 26, 2017 · 2 comments

Comments

@InternalLoss
Copy link

In the PHP implementation, if multiple recipients are specified, then the scripts simply just create the mail multiple times, wasting a lot of database space.

Example:

2017/11/26 15:50:57 MAIL FROM: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 RCPT TO: [email protected]
2017/11/26 15:50:57 DATA
2017/11/26 15:50:57 Date: 26 Nov 2017 21:39:16 -0000
2017/11/26 15:50:57 From: [email protected]
2017/11/26 15:50:57 To: [email protected],
2017/11/26 15:50:57  [email protected],
2017/11/26 15:50:57  [email protected],
2017/11/26 15:50:57  [email protected],
2017/11/26 15:50:57  [email protected],
2017/11/26 15:50:57  [email protected],
2017/11/26 15:50:57  [email protected],
2017/11/26 15:50:57  [email protected]

Would create 8 separate mail entries in the database, wasting space.
Tying in with #5, we could parse the mail and then have recipient ids in an indexable format where the database could select mails the user has not yet received, however once they have received it remove their Wii ID from the recipient list (for the sake of semantics you'd want to simply set them as sent so that the original mail contents are not edited but, to be honest, missing out all recipients except the ones that haven't yet got the mail shouldn't be an issue, it's not a security flaw).

@InternalLoss
Copy link
Author

@spotlightishere has this been solved in Mail-Go? This is a very old issue, I'm assuming it is.

@spotlightishere
Copy link
Contributor

As far as I can tell, this has not been dealt with in either implementation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants