[GitHub] [incubator-ponymail] sebbASF commented on issue #499: AAA regression in notifications API
sebbASF commented on issue #499: AAA regression in notifications API URL: https://github.com/apache/incubator-ponymail/issues/499#issuecomment-513545429 AFAICT list_raw is used in several places other than notifications. If the field is not present in some entries, then every access will potentially need to allow for this. Seems to me the obvious fix is for list_raw always to be present It seems a fundamental part of the schema design. This will require existing databases to be fixed, but that is a once-off. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-ponymail] Humbedooh opened a new issue #499: AAA regression in notifications API
Humbedooh opened a new issue #499: AAA regression in notifications API URL: https://github.com/apache/incubator-ponymail/issues/499 notifications.lua looks for notifications, and if AAA is enabled, tries to parse list-id which requires a `list_raw` element present. As that isn't set in archiver.py, it causes a nil error. Three possible ways of fixing: - fall back to using the `list` element if no list_raw exists (easiest, and best for backwards compat) - have archiver also add a `list_raw` to notifications - don't copy the email into the notifications db, but rather have a reference to the mbox document This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
