#6460 Investigate missing notification for openemr [ss4656]

v1.0.0
closed
General
2015-08-20
2013-07-15
Chris Tsai
No

[forge:site-support:#4656]

Hi,

Still noting that about 1 in every 5-10 forums posts are not being emailed to myself. Here is the initial ticket I had regarding this issue:
http://sourceforge.net/p/forge/site-support/3829/

Here's the first one I noticed so far:
http://sourceforge.net/p/openemr/discussion/202506/thread/c6899c17/#526d

These reports are from user: bradymiller

Discussion

  • Dave Brondsema

    Dave Brondsema - 2013-07-25
    • Milestone: forge-backlog --> forge-aug-09
     
  • Chris Tsai - 2013-07-25

    He notes another example: http://sourceforge.net/p/openemr/discussion/202506/thread/72aa6945/#efd1

    From ~7 hours ago, which I believe is outside of the range of the log disk space deal.

     
  • Dave Brondsema

    Dave Brondsema - 2013-07-26
    • Size: --> 2
     
  • Chris Tsai - 2013-07-26
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -8,3 +8,4 @@
     >Here's the first one I noticed so far:
     http://sourceforge.net/p/openemr/discussion/202506/thread/c6899c17/#526d
    
    +These reports are from user: bradymiller
    
     
  • Anonymous - 2013-07-27

    Originally by: bradymiller

    Here's another one that didn't get emailed out:
    http://sourceforge.net/p/openemr/discussion/202506/thread/778f857d/#4efe

     
  • Cory Johns - 2013-07-29
    • status: open --> in-progress
     
  • Cory Johns - 2013-07-29
    • status: in-progress --> code-review
     
  • Cory Johns - 2013-07-29

    allura:cj/6460

    Test case should cover it, but if you want to manually reproduce, shut down taskd, create notifications on two unrelated project with two non-site-admin users that don't have access to the other project, then start taskd and watch allura.log for the "Skipping notification - Userdoesn't have read access" message.

    Before the fix, one of the two notifications (depending which gets triggered first) will be skipped due to the user for that notification not having access to the project for the other notification whose project is in c.project

    After the fix, both should be dispatched.

     
    • QA: Tim Van Steenburgh
     
    • status: code-review --> closed
     

Log in to post a comment.