Smacks PubSub implementation has some flaws and is missing some higher level api stuff.
This is a non-exhaustive list of what is missing from the PubSub api:
Things that are broken:
- getNode sometimes throws NotALeafNodeException and it is not clear to the user what to do in that case
Thanks for pulling these together. I think I will be wanting to use all of the first three “missing” items for my application.
Indeed, I already implemented “modify node subscriptions” with a little test - though I have yet to test it with a real server, and I didn’t get set up on github yet to generate a pull request - then I got a little sidetracked by actual paid work
The other two look quite trivial to implement as well, so I would be happy to confront those too. So long as nobody’s in a big hurry !
Well, first you can expect a PR from me at some point for the Modify Node Subscriptions - just don’t hold your breath…
1 Like