| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \
| |/
|/| |
Don't "forget" listener options
|
|/
|
|
|
|
|
|
|
|
|
|
| |
If a listener is started or stopped via ejabberd_listener:add_listener/3
or ejabberd_listener:delete_listener/3, the configuration for all
listener modules is updated using the Module:transform_listen_option/2
function for each listener module that exports such a function.
However, for listener modules that don't provide that function (such as
ejabberd_stun), all but one option was dropped. This is now fixed.
The issue could be triggered e.g. by enabling mod_proxy65 in the modules
section.
|
| |
|
| |
|
|\
| |
| | |
Fix the list of dependencies
|
| |
| |
| |
| |
| |
| | |
The XEP-0227 code no longer uses exmpp.
Thanks to Steve Gillespie for noting this.
|
|/
|
|
| |
Thanks to Steve Gillespie for reporting the error.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| | |
XEP-0280: Don't send v1 carbon copies back to the sender
|
| |
| |
| |
| |
| | |
Use the existing is_carbon_copy/1 function, and combine multiple case
clauses into a single one.
|
| |
| |
| |
| |
| | |
The mod_carboncopy:is_carbon_copy/1 function now returns true not only
for <sent/>, but also for <received/> carbon copies.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
An earlier version of XEP-0280 specified the <received/> and <sent/>
tags to be siblings of the <forwarded/> element, whereas the current
version mandates them to be parents of <forwarded/>. The mod_carboncopy
module supports both variants. However, the check that makes sure
clients won't receive a copy of the messages they sent didn't work for
the old-style schema. This is now fixed.
|
| | |
|
| | |
|
|/ |
|
| |
|
| |
|
|\
| |
| | |
Log just one [info] message on successful XEP-0280 negotiation
|
|/
|
|
|
|
| |
Log one instead of three [info] messages when XEP-0280 (Message Carbons)
support is enabled or disabled successfully. On failure, log an
additional [warning].
|
|\
| |
| | |
XEP-0198: Cosmetic change: Reuse event handler
|
| |
| |
| |
| |
| |
| | |
On stanza queue overflow, pass a message to self() using the exclamation
mark operator instead of send_all_state_event/2. This allows for
reusing the existing handler for 'kick' events.
|
|\|
| |
| | |
XEP-0198: Terminate session if stanza queue becomes too large
|
|/
|
|
|
|
| |
On queue overflow, terminate the c2s session instead of just dropping
items from the queue. This makes sure all stanzas are either delivered
or bounced.
|
|\
| |
| | |
XEP-0198: Cosmetic change: Simplify state change
|
| |
| |
| |
| |
| |
| | |
When the FSM goes into the 'wait_for_resume' state, let fsm_next_state/2
take care of updating #state.mgmt_state and of writing the log line.
This doesn't change the behavior, but simplifies the code.
|
|\ \
| |/
|/| |
Let ejabberd_c2s always use fsm_next_state/2
|
|/
|
|
|
| |
Make sure any logic implemented in ejabberd_c2s:fsm_next_state/2 is
always applied.
|
|\
| |
| | |
XEP-0198: Don't exit on socket send failure
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If stream management is enabled, don't exit the c2s process when
ejabberd_socket:send/2 fails, but close the socket instead. This gives
the client a chance to resume the session.
Thanks go to Matthias Rieber for reporting the issue, providing detailed
logs, and testing the fix.
|
|\ \
| | |
| | | |
XEP-0198: Don't drop session on failed resume
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The 'previd' value provided by the client during a session resume
request includes the client's JID and ejabberd's session ID. If there
is a session for the requested JID but with a different session ID,
resumption should fail, but that session shouldn't be closed. This
commit makes sure the latter won't happen.
In practice, this will only make a difference in odd corner cases.
|
|\ \
| | |
| | | |
XEP-0198: Cosmetic change: Omit redundant guard
|
| |/
| |
| |
| |
| | |
The stream management state is never 'pending' when the c2s FSM is in
the 'session_established' state.
|
|\ \
| | |
| | | |
Omit XML namespace declaration for <iq/> stanzas
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Only the child elements of <iq/> stanzas are qualified by the namespaces
in question, not the <iq/> stanzas themselves.
This change just clarifies the code. It doesn't alter the behaviour, as
those <iq/> stanzas are handed over to jlib:iq_to_xml/1, and that
function ignores the 'xmlns' attribute anyway.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|