Closed (outdated)
Project:
Drupal core
Version:
7.x-dev
Component:
aggregator.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Aug 2008 at 16:40 UTC
Updated:
20 Mar 2020 at 16:52 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dries commentedI've committed this to CVS HEAD. The patch didn't apply to DRUPAL-6 so I did not commit it to Drupal 6. I'll lower the version module and mark it 'code needs work'.
Comment #2
catchProbably better as 'to be ported'.
Comment #3
mustafau commentedPatch for DRUPAL-6.
Comment #4
mustafau commentedWe only update URL if redirect is permanent (status 301). Previous patch called watchdog for 301, 302, 307.
Attached patch fixes the problem. Patch for DRUPAL-6 is also attached.
Comment #6
mustafau commentedPatch is fine. Testbed tried to apply patch for DRUPAL-6 to HEAD.
Comment #7
Anonymous (not verified) commentedThe last submitted patch failed testing.
Comment #8
kars-t commentedThe code for D7 is now:
This still does not check for status 301 but if the url did change. To me it still seems wise to change the code to check for http 301 and not the url but maybe someone does know better and close this issue / set it back to D6 which still applies?