The unsubscribe URL from the [comment-subscribed:unsubscribe-url] token needs to be an absolute URL in the notification emails. Please review the attached patch.
Original report
My watcher email wasn't working properly in that the comment unsubscribe URL was just showing as an absolute path without a protocol or domain.
You can stop receiving emails when someone replies to this post, by going to
/comment_notify/disable/7AV75zg-8gb4PWFkzzlJVPeGonak_wCdSXk3E9tXUvo
I pasted all the tokens into the body field to test them and received this in the email:
Unsubscribe URL /comment_notify/disable/7AV75zg-8gb4PWFkzzlJVPeGonak_wCdSXk3E9tXUvo The URL to disable notifications for the comment.
Absolute URL [comment-subscribed:unsubscribe-url:absolute] The absolute URL.
Arguments [comment-subscribed:unsubscribe-url:args] The specific argument of the current page (e.g. 'arg:1' on the page 'node/1' returns '1').
Brief URL [comment-subscribed:unsubscribe-url:brief] The URL without the protocol and trailing backslash.
Path [comment-subscribed:unsubscribe-url:path] The path component of the URL.
Relative URL [comment-subscribed:unsubscribe-url:relative] The relative URL.
Unaliased URL [comment-subscribed:unsubscribe-url:unaliased] The unaliased URL.
Comment | File | Size | Author |
---|---|---|---|
#5 | comment_notify-absolute-url-2911686-5.patch | 1.29 KB | ShaunDychko |
|
Comments
Comment #2
ShaunDychko CreditAttribution: ShaunDychko as a volunteer commentedComment #3
ShaunDychko CreditAttribution: ShaunDychko as a volunteer commentedComment #4
ShaunDychko CreditAttribution: ShaunDychko as a volunteer commentedUpdated the code comment for the @return annotation.
Comment #5
ShaunDychko CreditAttribution: ShaunDychko as a volunteer commentedRemove extraneous variable.
Comment #6
nkoporecHey @Shaun, tested your patch and it's working as expected.It removes the bug and set the absolute path.Good job!
Comment #8
gregglesThanks for the issue submission, patch, and review!