Skip to content

Commit

Permalink
Remove mapping of .php files to text/html
Browse files Browse the repository at this point in the history
`AddType test/html php` creates some problems¹ that can only be
resolved by adding more PHP specific² configs, and that is beyond
the scope of this project.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

¹ #62
² https://php.net/manual/ro/install.unix.apache2.php
  http://blog.famillecollet.com/post/2013/01/13/PHP-and-Apache-SetHandler-vs-AddHandler

Thanks @bridgeport!

Fix #62
  • Loading branch information
alrra committed Mar 3, 2015
1 parent 27348d0 commit daab35b
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 24 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
### HEAD

* Remove mapping of `.php` files to `text/html`
[[aeb81e9](https://github.com/h5bp/server-configs-apache/commit/aeb81e9174a9c69fa3a6f6c107d53dd4c8d58760)].

### 2.12.0 (March 2, 2015)

* Add `ServerSignature Off`
Expand Down
1 change: 0 additions & 1 deletion dist/.htaccess
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,6 @@ Options -MultiViews
AddType application/x-chrome-extension crx
AddType application/x-opera-extension oex
AddType application/x-xpinstall xpi
AddType text/html php
AddType text/vcard vcard vcf
AddType text/vnd.rim.location.xloc xloc
AddType text/vtt vtt
Expand Down
1 change: 0 additions & 1 deletion src/media_types/media_types.conf
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,6 @@
AddType application/x-chrome-extension crx
AddType application/x-opera-extension oex
AddType application/x-xpinstall xpi
AddType text/html php
AddType text/vcard vcard vcf
AddType text/vnd.rim.location.xloc xloc
AddType text/vtt vtt
Expand Down
1 change: 0 additions & 1 deletion test/fixtures/.htaccess
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,6 @@ Options -MultiViews
AddType application/x-chrome-extension crx
AddType application/x-opera-extension oex
AddType application/x-xpinstall xpi
AddType text/html php
AddType text/vcard vcard vcf
AddType text/vnd.rim.location.xloc xloc
AddType text/vtt vtt
Expand Down
10 changes: 0 additions & 10 deletions test/fixtures/test.php

This file was deleted.

11 changes: 0 additions & 11 deletions test/tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -308,17 +308,6 @@ exports = module.exports = {
}
},

'test.php': {
responseHeaders: {
'cache-control': 'max-age=0, no-transform',
'content-security-policy': "script-src 'self'; object-src 'self'",
'content-type': 'text/html; charset=utf-8',
'x-frame-options': 'DENY',
'x-ua-compatible': 'IE=edge',
'x-xss-protection': '1; mode=block'
}
},

'test.oex': {
responseHeaders: {
'content-encoding': null,
Expand Down

0 comments on commit daab35b

Please sign in to comment.