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
FAILED: [[SimpleTest]]: [MySQL] 190 pass(es), 0 fail(s), and 220 exception(s). 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

greggles’s picture

FileSize
1.39 KB
PASSED: [[SimpleTest]]: [MySQL] 36,512 pass(es). View
1.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
1.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.

greggles’s picture

Status: Active » Needs review

ahem.

Status: Needs review » Needs work

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

greggles’s picture

Status: Needs work » Needs review

The file that matters passes tests.

chx’s picture

Status: Needs review » Reviewed & tested by the community

Makes sense and does not even break string freeze.

catch’s picture

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.

greggles’s picture

FileSize
1.37 KB
PASSED: [[SimpleTest]]: [MySQL] 39,906 pass(es). View

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

dcam’s picture

greggles’s picture

Status: Needs review » Reviewed & tested by the community

Same RTBC as previous patch.

David_Rothstein’s picture

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.

dcam’s picture

Status: Patch (to be ported) » Needs review
FileSize
1.32 KB
FAILED: [[SimpleTest]]: [MySQL] 190 pass(es), 0 fail(s), and 220 exception(s). View

Backported #7 to D6.

greggles’s picture

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

greggles’s picture

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

Same RTBC as previous.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 11: 1564996-11-one-time-login-watchdog.patch, failed testing.

dcam’s picture

Issue summary: View changes
Status: Needs work » Reviewed & tested by the community

That failure is weird. D6 doesn't even have tests.

Status: Reviewed & tested by the community » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.