WordPress.org

Ready to get started?Download WordPress

Forums

Knews Multilingual Newsletters
[resolved] more valid email addresses (5 posts)

  1. ernscht
    Member
    Posted 1 year ago #

    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/

  2. creverter
    Member
    Plugin Author

    Posted 1 year ago #

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

    Regards,
    Carles.

  3. ernscht
    Member
    Posted 1 year ago #

    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)

  4. creverter
    Member
    Plugin Author

    Posted 1 year ago #

    Thank you very much Ernscht!

    We implement it in the next release...

    Regards!
    Carles.

  5. creverter
    Member
    Plugin Author

    Posted 1 year ago #

    Ok! This bug is solved in the 1.2.5 version.

    Thank you very much, Ernscht!!!

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic