New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filters need to check that they do not make URLs that are too long #181

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 4 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter

GoogleCodeExporter commented Apr 6, 2015

Branched off of Issue 149.

All filters need to make sure they don't make URLs (and URL segments) too long.

Original issue reported on code.google.com by sligocki@google.com on 7 Jan 2011 at 7:28

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

WIth the latest build / rpm this is still not fixed, is this correct ?

(36)File name too long: access to 
/wp-admin/load-scripts.php,qc=0,aload=hoverIntent,,common,,wp-ajax-response,,wp-
lists,,jquery-ui-sortable,,postbox,,thickbox,,jquery-ui-tabs,,jquery-color,,jque
ry-ui-resizable,,admin-comments,,dashboard,,plugin-install,,media-upload,aver=fa
13a2240f94d7c3b48b3fa339f2a4f0.pagespeed.jm.3FJMWmJn7I.js failed

Original comment by pdreis...@gmail.com on 10 Jan 2011 at 12:38

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

Correct.  It was only fixed for the case you reported earlier in Issue 149 and 
we will do a broader fix for this issue.

Original comment by jmara...@google.com on 10 Jan 2011 at 1:48

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

Note also http://code.google.com/p/modpagespeed/issues/detail?id=176 which, 
when implemented, should reduce the frequency of this problem considerably.

Original comment by jmara...@google.com on 14 Jan 2011 at 6:58

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

This should be fixed in trunk as of r601.

Original comment by sligocki@google.com on 30 Mar 2011 at 10:11

  • Changed state: Fixed
  • Added labels: release-note
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment