WordPress.org

Support

Support » Plugins and Hacks » [Resolved] [Plugin: Knews Multilingual Newsletters] more valid email addresses

[Resolved] [Plugin: Knews Multilingual Newsletters] more valid email addresses

  • some valid email addresses fails in the confirm & unsubscribe-process.
    e.g. abc+def@test.com

    urlencoding the email addresses in all emails would possibly fix the problem.

    a) knews.php
    $url_confirm = KNEWS_LOCALIZED_ADMIN . ‘admin-ajax.php?action=knewsConfirmUser&k=’ . $confkey . ‘&e=’ . urlencode($email);

    b) knews_admin_news_send.php
    … ‘admin-ajax.php?action=knewsUnsubscribe&e=’ . urlencode($user->email) . ‘&k=’ . $user->confkey;

    c) …

    http://wordpress.org/extend/plugins/knews/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author creverter
    Member

    @creverter

    Thanks Ernscht! I just released a new version, and now urlencode / decode email param, this should solve the problem.

    Regards,
    Carles.

    thanks carles for the fast fix

    but in my installation there is still the same problem
    the confirmation only works without the urldecoding part.
    knew.php:
    eg.
    function confirm_user_self() {

    //$email = mysql_real_escape_string(urldecode($_GET[‘e’]));
    $email = mysql_real_escape_string($_GET[‘e’]);

    (the $_GET variables are already decoded)

    Plugin Author creverter
    Member

    @creverter

    Thank you very much Ernscht!

    We implement it in the next release…

    Regards!
    Carles.

    Plugin Author creverter
    Member

    @creverter

    Ok! This bug is solved in the 1.2.5 version.

    Thank you very much, Ernscht!!!

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘[Resolved] [Plugin: Knews Multilingual Newsletters] more valid email addresses’ is closed to new replies.