Opened 10 years ago
Closed 10 years ago
#28766 closed enhancement (invalid)
Permalinks Options/Notices on nginx + HHVM incorrect
Reported by: | crxgames | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 3.9 |
Component: | Permalinks | Keywords: | has-patch needs-testing |
Focuses: | Cc: |
Description
The permalink settings page in the admin panel is incorrect when running Wordpress under HHVM + nginx. index.php is always included in the permalink samples and the notice messages are those from Apache.
This patch adds a new global, $is_hhvm, in the same fashion nginx is tested for, and updates the permalinks settings page accordingly.
Attachments (1)
Change History (5)
#2
@
10 years ago
- Focuses administration removed
- Keywords has-patch needs-testing added
- Milestone changed from Awaiting Review to Future Release
- Type changed from defect (bug) to enhancement
#3
@
10 years ago
So, should we report this as an HHVM issue instead of include a work around for it perhaps?
#4
@
10 years ago
- Milestone Future Release deleted
- Resolution set to invalid
- Status changed from new to closed
This was fixed in HHVM 3.6.1.
Note: See
TracTickets for help on using
tickets.
Thanks for the report crxgames.
Interesting that HHVM doesn't pass the web server name through to
$_SERVER['SERVER_SOFTWARE']
. I would have expected it to pass through 'nginx' if I'm running Nginx.What happens if you're running HHVM behind Apache instead of Nginx? The return value of
got_url_rewrite()
will return true even ifgot_mod_rewrite()
returns false. (Granted, if you're running HHVM you're unlikely to be running Apache without mod_rewrite, but the point still stands.)