Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2705)

Issue 7915: [PATCH] SendFeatures hook for additions to stream:features

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 5 months ago by Jacob
Modified:
9 years, 4 months ago
Reviewers:
djabberd
Base URL:
http://code.sixapart.com/svn/djabberd/trunk/DJabberd
Visibility:
Public.

Description

It seems there are some methods on VHost for adding features, but these are intended for us in responding to the http://jabber.org/protocol/disco IQ. A change is needed to allow plugin to insert things into the <features> stanza send after stream start from server to client. Currently, this stanza might look something like: <stream:features><auth xmlns='http://jabber.org/features/iq-auth'/><starttls xmlns='urn:ietf:params:xml:ns:xmpp-tls' /></stream:features>

Patch Set 1 #

Patch Set 2 : A hook for enabling plugins to insert into features stanza #

Unified diffs Side-by-side diffs Delta from patch set Stats (+147 lines, -1 line) Patch
lib/DJabberd/Connection.pm View 1 1 chunk +11 lines, -0 lines 0 comments Download
lib/DJabberd/HookDocs.pm View 1 chunk +7 lines, -0 lines 0 comments Download
t/features-hook.t View 1 chunk +124 lines, -0 lines 0 comments Download
t/lib/djabberd-test.pl View 1 chunk +5 lines, -1 line 0 comments Download

Messages

Total messages: 3
Jacob
15 years, 5 months ago (2008-11-03 19:10:32 UTC) #1
dangabrad
Assuming all tests still pass, looks good to me. Feel free to commit.
15 years, 5 months ago (2008-11-04 06:56:59 UTC) #2
Jacob
15 years, 5 months ago (2008-11-10 22:02:20 UTC) #3
A hook for enabling plugins to insert into features stanza
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b