1351695 - Allow apache to ignore .htaccess for increased performance

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: General, enhancement)

Description

[Dylan Hardison [:dylan] (he/him)]

if you run strace on httpd during normal operation, you'll see we make thousands of stat() calls to .htaccess files.
One request for a file under /data/assets is three calls to stat(), and if NFS is involved those calls can take a decent amount of time.

We don't need to do this, we already send configuration directives to apache from mod_perl.

I intend to make mod_perl.pl read in the .htaccess file, and send the configuration to apache and also send AllowOverride None, which will prevent apache from looking for .htaccess files.

Comment 1

[Dylan Hardison [:dylan] (he/him)]

Attachment: 1351695_1.patch

Comment 2

[:glob ✱]

Comment on attachment 8852524 [details] [diff] [review]
1351695_1.patch

Review of attachment 8852524 [details] [diff] [review]:
-----------------------------------------------------------------

as per irc, there's multiple .htaccess files to accommodate:
https://github.com/mozilla-bteam/bmo/blob/master/Bugzilla/Install/Filesystem.pm#L340

Comment 3

[Dylan Hardison [:dylan] (he/him)]

Attachment: 1351695_2.patch

I moved this to its own file as it is easier to validate outside of apache:

perl -T -I. -MBugzilla::ModPerl -E 'say Bugzilla::ModPerl->apache_config(".")'

and it now looks for all .htaccess files.

Comment 4

[Dylan Hardison [:dylan] (he/him)]

Attachment: 1351695_3.patch

sans Alive logging code.

Comment 5

[:glob ✱]

Comment on attachment 8852593 [details] [diff] [review]
1351695_3.patch

Review of attachment 8852593 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob

::: Bugzilla/ModPerl.pm
@@ +5,5 @@
> +#
> +# This Source Code Form is "Incompatible With Secondary Licenses", as
> +# defined by the Mozilla Public License, v. 2.0.
> +
> +package Bugzilla::ModPerl;

a package so good it was declared twice.

this may be better named Bugzilla::ModPerlConfig as that's all it does.

@@ +17,5 @@
> +
> +# We don't need (or want) to use Bugzilla's template subclass.
> +use Template ();
> +
> +sub apache_config {

a comment about why this is necessary would be helpful here.

@@ +28,5 @@
> +    my $wanted = sub {
> +        package File::Find;
> +        our ( $name, $dir );
> +
> +        if ( $name =~ /\.htaccess$/ ) {

$name =~ m#/\.htaccess$#

Comment 6

[Dylan Hardison [:dylan] (he/him)]

With corrections

To [e-mail]:mozilla-bteam/bmo.git
   b2744a5..30c35b3  master -> master