Currently when a user uses their one-time login link there is a watchdog message associated with uid 0 but with text that lets us know which uid/name used the link. This is fun for visual review but pretty annoying if you're, say, trying to audit which users are logging in with the one-time link vs. the normal form.

This teeny tiny patch moves the watchdog call a few lines to make it way more useful.

Files: 
CommentFileSizeAuthor
#11 1564996-11-one-time-login-watchdog.patch1.32 KBdcam
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]
#7 1564996_one_time_watchdog_more_info.patch1.37 KBgreggles
PASSED: [[SimpleTest]]: [MySQL] 39,906 pass(es).
[ View ]
#1 1564996_one_time_watchdog_more_info-D6.patch1.32 KBgreggles
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1564996_one_time_watchdog_more_info-D6.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#1 1564996_one_time_watchdog_more_info-D7.patch1.37 KBgreggles
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1564996_one_time_watchdog_more_info-D7.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#1 1564996_one_time_watchdog_more_info-D8.patch1.39 KBgreggles
PASSED: [[SimpleTest]]: [MySQL] 36,512 pass(es).
[ View ]

Comments

StatusFileSize
new1.39 KB
PASSED: [[SimpleTest]]: [MySQL] 36,512 pass(es).
[ View ]
new1.37 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1564996_one_time_watchdog_more_info-D7.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
new1.32 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1564996_one_time_watchdog_more_info-D6.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

And patches.

Status:Active» Needs review

ahem.

Status:Needs review» Needs work

The last submitted patch, 1564996_one_time_watchdog_more_info-D6.patch, failed testing.

Status:Needs work» Needs review

The file that matters passes tests.

Status:Needs review» Reviewed & tested by the community

Makes sense and does not even break string freeze.

Version:8.x-dev» 7.x-dev
Status:Reviewed & tested by the community» Needs review
Issue tags:+needs backport to D6, +needs backport to D7

Makes sense. Committed/pushed to 8.x.

StatusFileSize
new1.37 KB
PASSED: [[SimpleTest]]: [MySQL] 39,906 pass(es).
[ View ]

Same patch with a new name, hoping the d7 testbot will like this more.

Status:Needs review» Reviewed & tested by the community

Same RTBC as previous patch.

Version:7.x-dev» 6.x-dev
Category:feature» task
Status:Reviewed & tested by the community» Patch (to be ported)

Makes sense - seems like more of a bug fix than a feature request actually. Let's split the difference and call it a task :)

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/bbf0134

This is marked as "needs backport to D6" so I'll move it back there, in case someone wants to work on it. I don't think non-security patches have been committed to Drupal 6 for a long time, though.

Status:Patch (to be ported)» Needs review
StatusFileSize
new1.32 KB
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]

Backported #7 to D6.

FWIW, I do consider this to be a security improvement.

Assigned:greggles» Unassigned
Status:Needs review» Reviewed & tested by the community

Same RTBC as previous.