#3985 closed defect (bug) (fixed)
Out of bounds pagination requests to the plugin directory return a 'raw' 404
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | Priority: | low | |
| Component: | Plugin Directory | Keywords: | seo |
| Cc: |
Description
E.g., https://wordpress.org/plugins/browse/popular/page/999/ returns a raw NGINX 404 page.
These should return standard WP 404 behaviour.
Change History (9)
#2
follow-up:
↓ 3
@
6 years ago
Nope, still need https://wordpress.org/plugins/browse/popular/page/100/ etc to return a 'nice' 404.
#3
in reply to:
↑ 2
@
6 years ago
- Keywords reporter-feedback removed
Replying to jonoaldersonwp:
Nope, still need https://wordpress.org/plugins/browse/popular/page/100/ etc to return a 'nice' 404.
That won't be possible, as it's blocked at the load balancer. As these pages are not linked to (if they are, let us know), and are only accessible via manual link creation I don't see any need to push for a change there. We could instead have the blocked pages redirect to /plugins/ maybe, but it really doesn't seem worth it.
I'm willing to close this as good-enough.
#4
@
6 years ago
I'm not keen on writing this off. Anybody can link to those pages, and external systems (Google, Facebook etc), can, and will, freely request them.
Frankly, that any URL at all on the domain(s) can return a 'raw' error page is far from 'good-enough', and we should address that as a serious SEO/UX/analytics issue. Error states should never redirect, either, for reference.
If we're serving that error via the load balancer, at the very least, we could at the very least serve a cleaner/nicer static HTML page which contained useful content and resource links, appropriate meta (and HTTP header) content, and our Google Tag Manager code.
That said, if we're already routing these requests via the load-balancer, why can't we route them to a native WP 404 state (i.e., rewrite the request invisibly to wordpress.org/error)?
Re-opening!
This ticket was mentioned in Slack in #meta by jonoaldersonwp. View the logs.
6 years ago
#6
@
6 years ago
- Keywords pending-systems added
- Owner set to dd32
- Status changed from new to accepted
I've made a systems request for this: https://make.wordpress.org/systems/2019/01/08/as-per-https-meta-trac-wordpress-org-ticket-3985-can-we/
This was initially reported and fixed in #496.
The nginx rule is set up on load balancers to avoid excess load caused by the high pages.
The list of pages on https://wordpress.org/plugins/browse/popular/ is now limited to 99, any page beyond that should not be linked to from anywhere. I guess that resolves the issue?