• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

processone / ejabberd / 807
34%

Build:
DEFAULT BRANCH: master
Ran 13 Sep 2024 07:35PM UTC
Jobs 1
Files 289
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

13 Sep 2024 06:50PM UTC coverage: 32.985% (+0.001%) from 32.984%
807

push

github

weiss
mod_pubsub: Don't blindly echo PEP notification

Since commit 514c25cae, whenever a PEP
item was published, a notification was blindly sent back to the owner.
However, this should only be done subject to +notify filtering, as per
XEP-0163:

| the PEP service shall send notifications to all of the account owner's
| available resources (subject to notification filtering).

The motivation for the mentioned commit was that +notify subscriptions
initially didn't work for PEP node owners (#2108).  However, that issue
was fixed by commit 5968bc931 (#2112).
As a result, the owner's client was actually notified twice if the
client was subscribed to the node (e.g., via +notify).

Therefore, just omit the additional, blind notification.

Thanks to W. Martin Borgert and Daniel Gultsch for reporting the issue.

14531 of 44054 relevant lines covered (32.98%)

617.26 hits per line

Jobs
ID Job ID Ran Files Coverage
1 807.1 13 Sep 2024 07:35PM UTC 289
32.98
GitHub Action Run
Source Files on build 807
  • Tree
  • List 289
  • Changed 48
  • Source Changed 2
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • 3469a51f on github
  • Prev Build on master (#806)
  • Next Build on master (#808)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc