801928 - Redirect behavior on our staging env is different than what is on production

Status: RESOLVED FIXED

(Webtools :: Bouncer, defect)

Description

[Matt Brandt [:mbrandt]]

When passing arguments for an OS and lang that do not exist the redirect behavior on stage is different than what is occurring on production.
- Stage 404s but redirects to SSL
- Production 404s but maintains it's non-SSL sheme

Steps to reproduce:
1. goto 
   - http://download.allizom.org/?lang=kitty_language&product=firefox-16.0b6&os=stella
   - http://download.allizom.org/?lang=kitty_language&product=firefox-16.0b6&os=stella
2. observe their behavior

Actual:
- [15:48:03.534] GET https://download.allizom.org/?lang=kitty_language&product=firefox-16.0b6&os=stella [HTTP/1.0 404 Not Found 81ms]

- [15:49:06.401] GET http://download.mozilla.org/?lang=kitty_language&product=firefox-16.0b6&os=stella [HTTP/1.0 404 Not Found 36ms]

Suggested:
Can we make the behavior consistent between environments? This will aid in simplifying QA's automated tests.

Comment 1

[Matt Brandt [:mbrandt]]

This is still occurring, can we get this remedied?

Comment 2

[[:AndreiH]]

Is this still under investigation? Our automated tests are skiped on stage for some time now.

Comment 3

[Robert Helmer [:rhelmer]]

(In reply to [:AndreiH] from comment #2)
> Is this still under investigation? Our automated tests are skiped on stage
> for some time now.

I just took over bouncer for Brandon, have not had a chance to take a look quite yet. I do see the same behavior as comment 0 still, I suspect it's a configuration issue either in Apache, the load balancer, or bouncer itself.

:cturra, would it be possible to share the above configs with me for both prod and stage ^? I can probably figure out what the discrepancy here is - would be good to familiarize myself, anyway. Thanks!

Comment 4

[Chris Turra [:cturra]]

the following are the apache configs for each environment as requested:

stage:
---
<VirtualHost *:80>
    ServerName download.allizom.org
    Redirect 301 / https://download.allizom.org/
</VirtualHost>

<VirtualHost *:81>
    ServerName download.allizom.org
    ServerAlias download.mozilla.org # QA /etc/hosts testing
    DocumentRoot /data/www/download.allizom.org/tuxedo/bouncer/php
    SetEnvIf SSLSessionID .+ HTTPS=on

    <Directory "/data/www/download.allizom.org/tuxedo/bouncer/php">
      Options +FollowSymLinks
      AllowOverride All
    </Directory>

    LogLevel debug
    LogFormat "%h %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\"" combined
    ErrorLog "|/usr/sbin/rotatelogs /var/log/httpd/download.mozilla.org/error_log_%Y-%m-%d-%H 3600 -0"
    CustomLog "|/usr/sbin/rotatelogs /var/log/httpd/download.mozilla.org/access_%Y-%m-%d-%H 3600 -0" combined

    RewriteEngine on
    # Bug 932273 - Please update download links for EUballot for Fx 25.0
    RewriteCond %{QUERY_STRING} ^(.*)product=firefox-latest-euballot(.*)$
    RewriteRule /(.*)$ http://download-eu.mozilla.org/$1?%1product=firefox-25.0-euballot%2 [R=302,L]

    # bug 860274
    RewriteCond %{QUERY_STRING} product=firefox-20\.0&os=win(.*)$ [NC]
    RewriteRule (.*) $1?product=firefox-20\.0\.1&os=win%1 [L,R=301]
</VirtualHost>




prod:
---
<VirtualHost *:80>
    ServerName download.mozilla.org
    Redirect 301 / https://download.mozilla.org/
</VirtualHost>

<VirtualHost *:81>
    ServerName download.mozilla.org
    ServerAlias download-eu.mozilla.org
    DocumentRoot /data/www/download.mozilla.org/tuxedo/bouncer/php
    SetEnvIf SSLSessionID .+ HTTPS=on

    <Directory "/data/www/download.mozilla.org/tuxedo/bouncer/php">
      Options +FollowSymLinks
      AllowOverride All
    </Directory>

    <Directory /data/www/download.mozilla.org/tuxedo/bouncer/php>
        Options +FollowSymLinks
        AllowOverride None
    </Directory>

    ServerAdmin [e-mail]

    RewriteEngine on
    RewriteRule /pub(.*)    http://ftp.mozilla.org/pub/mozilla.org  [R=permanent]
    RewriteRule /admin/(.*)  https://bounceradmin.mozilla.com/admin/$1    [R=permanent]

    # bug 860274
    RewriteCond %{QUERY_STRING} product=firefox-20\.0&os=win(.*)$ [NC]
    RewriteRule (.*) $1?product=firefox-20\.0\.1&os=win%1 [L,R=301]

    CookieTracking on
    CookieName dmo
    CookieExpires "1 year"

    php_value memory_limit 64M

    # note the user entry is removed, and the cookie item is added
    LogFormat "%h %l %{cookie}n %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\"" cookie
    LogFormat "%h %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\"" combined
    CustomLog "|/usr/sbin/rotatelogs /var/log/httpd/download.mozilla.org/access_%Y-%m-%d-%H 3600 -0" combined
    ErrorLog "|/usr/sbin/rotatelogs /var/log/httpd/download.mozilla.org/error_log_%Y-%m-%d-%H 3600 -0"
    php_value newrelic.appname "download.mozilla.org"
</VirtualHost>

Comment 5

[Robert Helmer [:rhelmer]]

I believe this is fixed now - please reopen if not