Ticket #1484040 (closed Bugs: worksforme)

Opened 4 years ago

Last modified 3 years ago

Rename inc files to inc.php

Reported by: sts Owned by:
Priority: 6 Milestone:
Component: Security issue Version: 0.1-beta
Severity: minor Keywords:
Cc:

Description

As webservers are usually configured to display inc files as plain text documents and the program source is visible to everyone on the net, I think its extremly important to rename all .inc files to .inc.php.

A user already requested this at Bug #1253473, but only for configuration files. I think this is as well an important bug, before using roundcube on production systems.

Change History

Changed 4 years ago by lancey

  • status changed from new to closed
  • resolution set to worksforme
  • severity changed from major to minor

This is addressed in the .htaccess file, and should work for all apache users.

<FilesMatch "(\.inc|\~)$|^_">
  Order allow,deny
  Deny from all
</FilesMatch>

Renaming files to .inc.php won't allow them to be viewed, but will allow them to be executed - also not as safe as it should be.

The right thing is to just warn in the install instructions - everyone using webserver other than apache should set it up to deny serving .inc files.

Note: See TracTickets for help on using tickets.