Since vast majority of average users may get confused once they verify their email address by opening a bit.ly link and they may even freak out completely once they see a blank page, it would be much more intuitive to provide a closure (the last step of the process) with at least some themeable content with a default like the following text:

Your email address has been verified. To unsubscribe from our list anytime in the future, please save the following link: http://bit.ly/something Thank you!

Edit: I've noticed there are numerous of drupal_set_message and drupal_goto in this module in order to show some content on this page, but I'm not getting anything in there. I assigned a "Total Subscription Block" to my custom node type and the URL from the bit.ly looks like this:

http://mydomain/total-subscription-verify?&email=username@domain&entity_type=node-1&entity_id=134&tids=[]&POŠALJI=d67d8ab4f4c10bf22aa353e27879133c

Comments

jazzitup created an issue. See original summary.

jazzitup’s picture

Issue summary: View changes

Even the page is blank, the email address has been verified though...

jazzitup’s picture

Title: Show some content for "total-subscription-verify" » Issue when dealing with translated Subscribe/Unsubscribe values
Issue summary: View changes
Status: Active » Patch (to be ported)
StatusFileSize
new853 bytes

I have submitted a patch. The issue was related to my non-English Drupal setup, therefore Subscribe/Unsubscribe values were also translated and didn't match expectation from the hardcoded _GET variable in English.

NOTE: Even this issue is resolved temporarily, I'm afraid the issue may be still open for somebody else - since not all translated values out there are URL-friendly. We should do something about this. To improve the security and privacy I would recommend the following improvements:

  • avoid passing translated values via _GET vars
  • encode user's email address
deepakaryan1988’s picture

@jazzitup I think I have to look up on this issue.

deepakaryan1988’s picture

Status: Patch (to be ported) » Needs review

Status: Needs review » Needs work
deepakaryan1988’s picture

Issue tags: +Needs reroll, ++Needs reroll, +Need reroll

rishikant05’s picture

Assigned: Unassigned » rishikant05
Status: Needs work » Needs review
StatusFileSize
new853 bytes

rerolled #3
Please check.

Status: Needs review » Needs work
madhavvyas’s picture

Issue tags: -+Needs reroll, -Need reroll
deepakaryan1988’s picture

Assigned: rishikant05 » Unassigned
Status: Needs work » Fixed

Pushed it to latest dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.