-
Notifications
You must be signed in to change notification settings - Fork 426
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
C2s/pubsub #3785
C2s/pubsub #3785
Conversation
https://datatracker.ietf.org/doc/html/rfc6120#section-8.1.2.1 1. When a server receives an XML stanza from a connected client, the server MUST add a 'from' attribute to the stanza or override the 'from' attribute specified by the client, where the value of the 'from' attribute MUST be the full JID (<localpart@domainpart/resource>) determined by the server for the connected resource that generated the stanza (see Section 4.3.6), or the bare JID (<localpart@domainpart>) in the case of subscription-related presence stanzas (see [XMPP-IM]).
small_tests_24 / small_tests / e555aae ldap_mnesia_24 / ldap_mnesia / e555aae small_tests_25 / small_tests / e555aae ldap_mnesia_25 / ldap_mnesia / e555aae dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / e555aae pgsql_mnesia_24 / pgsql_mnesia / e555aae internal_mnesia_25 / internal_mnesia / e555aae dynamic_domains_pgsql_mnesia_25 / pgsql_mnesia / e555aae dynamic_domains_mysql_redis_25 / mysql_redis / e555aae elasticsearch_and_cassandra_25 / elasticsearch_and_cassandra_mnesia / e555aae dynamic_domains_mssql_mnesia_25 / odbc_mssql_mnesia / e555aae pgsql_mnesia_25 / pgsql_mnesia / e555aae riak_mnesia_24 / riak_mnesia / e555aae mysql_redis_25 / mysql_redis / e555aae mssql_mnesia_25 / odbc_mssql_mnesia / e555aae |
Codecov ReportBase: 64.42% // Head: 68.07% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## feature/mongoose_c2s #3785 +/- ##
========================================================
+ Coverage 64.42% 68.07% +3.64%
========================================================
Files 536 536
Lines 35002 35005 +3
========================================================
+ Hits 22551 23828 +1277
+ Misses 12451 11177 -1274
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
Really what needed fixing was the
from
verification in the core c2s code, that enabled not only pubsub but also another bunch of suites.