#296 mod_pubsub: Add "delete-nodes" and correct "create-nodes"

Reporter serkejal
Owner MattJ
Stars ★★★ (3)  
  • mod_pubsub
  • Type-Enhancement
  • Interoperability
  • Priority-Medium
  • Milestone-0.9
  • Status-Fixed
  1. serkejal on

    Hi all, i modify pubsub.lua and mod_pubsub.lua to support "delete-nodes" (it's mandatory), see http://xmpp.org/extensions/xep-0060.html#owner-delete; and to follow the standard for http://xmpp.org/extensions/xep-0060.html#owner-create; in the code there is a work-around to send notification of "delete-nodes" to subscribers...i work on it! i hope it could be of help!

  2. serkejal on

    Some test...some bugfix

  3. MattJ on

    Thanks! Currently all focus is on the 0.9 release (which is feature-frozen), but I'll take a look at these as soon as it's out and I have a little more time. Pubsub is something one area I want to bring up to scratch with the next release.

    • tags Status-Accepted
  4. MattJ on

    • title mod_pubsub: Add "delete-nodes" and correct "create-nodes"
    • tags Type-Defect Compliance Type-Enhancement Milestone-0.9.1 Interoperability
  5. BioShock78 on

    Hi, i install your plugin in /usr/lib64/prosody/modules (pubsub.lua and mod_pubsub.lua) and i have error on prosody.log, this is it: Oct 14 18:34:57 modulemanager error Error initializing module 'pubsub' on 'jabbi.pl': /usr/lib64/prosody/modules/mod_pubsub.lua:3: attempt to call global 'module' (a table value) stack traceback: /usr/lib64/prosody/core/modulemanager.lua:35: in function 'module' /usr/lib64/prosody/modules/mod_pubsub.lua:3: in main chunk (tail call): ? [C]: in function 'xpcall' /usr/lib64/prosody/core/modulemanager.lua:35: in function 'pcall' /usr/lib64/prosody/core/modulemanager.lua:129: in function 'load' /usr/lib64/prosody/core/modulemanager.lua:88: in function '?' /usr/lib64/prosody/util/events.lua:67: in function 'fire_event' /usr/lib64/prosody/core/hostmanager.lua:84: in function 'activate' /usr/lib64/prosody/core/hostmanager.lua:42: in function '?' /usr/lib64/prosody/util/events.lua:67: in function 'fire_event' /usr/lib64/prosody/../../bin/prosody:374: in function 'prepare_to_start' /usr/lib64/prosody/../../bin/prosody:490: in main chunk [C]: ? My english is bad , sorry. Please fast reply.

  6. serkejal on

    to answer quickly. Returns an error on line 3, that is : local jid_bare = require "util.jid".bare; does not seem to be a problem with "the code" but with configuration/installation of prosody What version of prosody are you using? Have you cp mod_pubsub.lua to /usr/lib64/prosody/modules and pubsub.lua to /usr/lib64/prosody/util/?

  7. serkejal on

    this is the last version..

  8. BioShock78 on

    Ok, thanks for the quick response. I did just like you wrote the new version. All kind of work, I added the component "pubsub.jabbi.pl" "PubSub" and search transports it appears. Jappix However, I do not want to work with him, crashes this error: Oct 15 7:50:39 p.m. boshef462279-eaef-47d1-a03a-608834199a2c warn rid of too large (means the request was lost). Last rid: 687,059 New rid: 687061 The second issue, however, is vjud, prosody no crashes no errors, but transport is not seen in the search box, write 'damaged transport'. Maybe you have a revised version of this?

  9. serkejal on

    About to vjud and Jappix i can't help you, i never used them but in my experience "warn rid of too large (means the request was lost)" refers to a BOSH request malformed or missed: The last rid (request id) is out of sequence, this can be a warning and not a problem, if the difference between two rid is not so high the request is accepted without error. (more info here http://xmpp.org/extensions/xep-0124.html#rids)

  10. BioShock78 on

    check this http://code.google.com/p/lxmppd/issues/detail?id=315&sort=-id&colspec=ID%20Type%20Status%20Priority%20Difficulty%20Milestone%20Owner%20Summary

  11. MattJ on

    • tags Milestone-0.9.1 Milestone-0.10
  12. Zash on

    • tag mod_pubsub
  13. Zash on

    Huh, what's this? https://hg.prosody.im/0.9/rev/518d864b2ab8

    • tags Milestone-0.9 Status-Fixed

New comment