sebbASF commented on issue #499: AAA regression in notifications API
URL: 
https://github.com/apache/incubator-ponymail/issues/499#issuecomment-513917244
 
 
   According to the  setup.py code, the notifications doc type is not set up 
with the list_raw attribute, which is presumably why the archiver does not try 
to populate it when saving the notifications.
   
   However, lists.a.o does populate the list_raw field for private emails 
(presumably the code that handles private mails is using a different version of 
the archiver).
   
   This may explain why the bug has only recently been noticed. (In my case, I 
only recently used the reply function for a public list reply)
   
   The simplest fix would be for notifications.lua to unconditionally populate 
list_raw from list.
   I don't see the point in checking to see if list_raw already exists.
   
   I'm not sure it's worth changing the notifications doc type to keep only a 
reference to the mbox document. That would mean fetching the mbox entry in 
order to check access.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to