-
Notifications
You must be signed in to change notification settings - Fork 428
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
Reduce stringprepping in mod_offline #3009
Conversation
This comment has been minimized.
This comment has been minimized.
55c83a3
to
a11ddff
Compare
This comment has been minimized.
This comment has been minimized.
a11ddff
to
c51b00c
Compare
get_max_user_messages gets its parameters from the #offline_msg record, which when created took the jid:to_lus/1 from a full jid, hence its arguments are already stringprepped. resend_offline_message_packet will forward the server parameter down the stack up to the timestamp function, which needs the stringprepped parameter, but this was stringprepped already before, so this processed version can be passed instead.
c51b00c
to
b639c66
Compare
9026.1 / Erlang 23.0.3 / small_tests / dcfc896 9026.2 / Erlang 23.0.3 / internal_mnesia / dcfc896 9026.4 / Erlang 23.0.3 / mysql_redis / dcfc896 9026.3 / Erlang 23.0.3 / odbc_mssql_mnesia / dcfc896 9026.7 / Erlang 23.0.3 / elasticsearch_and_cassandra_mnesia / dcfc896 9026.6 / Erlang 23.0.3 / ldap_mnesia / dcfc896 9026.5 / Erlang 23.0.3 / riak_mnesia / dcfc896 9026.9 / Erlang 22.3 / pgsql_mnesia / dcfc896 |
Codecov Report
@@ Coverage Diff @@
## master #3009 +/- ##
==========================================
- Coverage 79.15% 79.14% -0.01%
==========================================
Files 377 377
Lines 32763 32751 -12
==========================================
- Hits 25933 25921 -12
Misses 6830 6830
Continue to review 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.
Great work!
get_max_user_messages gets its parameters from the #offline_msg record,
which when created took the jid:to_lus/1 from a full jid, hence its
arguments are already stringprepped.
resend_offline_message_packet will forward the server parameter down the
stack up to the timestamp function, which needs the stringprepped
parameter, but this was stringprepped already before, so this processed
version can be passed instead.