When sending a deeplink by email from a node page, the resulting link does not evaluate to the actual node page. This occurs when the expiration time is set to 0.

Steps to reproduce:

  1. Send deeplink by email from a node page
  2. Click resulting link in email
  3. Access denied!
CommentFileSizeAuthor
#2 deeplink-8249169-access-denied-error-fix.patch832 byteseljefejb

Comments

eljefejb’s picture

eljefejb commented
Title: Deeplink by email results in Page Not Found error » Deeplink by email results in Access denied error
Issue summary: View changes

I believe this happens because, when the link is clicked, the deeplinks database entry is being deleted before the node is loaded. Looking at the database, it seems that the expires field is not being entered with a value of 0. Instead, it is being set to the current timestamp + the number of hours specified for expiration. The attached patch fixes the issue.

eljefejb’s picture

eljefejb commented
StatusFileSize
new deeplink-8249169-access-denied-error-fix.patch832 bytes
eljefejb’s picture

eljefejb commented
Component: Miscellaneous » Code

Oops, looks like I used the comment ID instead of the Issue ID in the patch name. Mea culpa. :)

eljefejb’s picture

eljefejb commented
Status: Active » Needs review