608361 - Loading Top Crashers By in one product populates the Versions pulldown with Firefox version numbers

Status: VERIFIED FIXED

(Socorro :: General, task)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

STR:

1. Load http://crash-stats.stage.mozilla.com/products/Fennec
2. Choose "Top Crashers By [domain], [url]" etc.
3. Now, look at the Version pulldown

Expected Results:

Fennec version numbers

Actual Results:

Firefox version numbers

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Possibly related fallout?  http://crash-stats.stage.mozilla.com/daily?p=Fennec&v[]=4.0b1 populated Product itself with Firefox, and very clearly pulls Firefox data.

Comment 2

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

The issue in https://bugzilla.mozilla.org/show_bug.cgi?id=608361#c1 is a separate bug, caused by Fennec not being found in config/daily.php.  Spinning off Bug 608383 for that.

Comment 3

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

In #0, the product also changes to Firefox.

Comment 4

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Okay, this was caused by moving to database session storage in Bug 608023.  Code is looking to access the selected product specifically from cookie; I'll look to see if there is any other code that might be affected as well.

Comment 5

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

After further investigation, it turns out that this issue is actually being caused by setting the value in config/cookie.php for $config['secure'] to TRUE in Bug 607081. Once $config['secure'] is changed to FALSE, the problem listed above no longer happens.

Comment 6

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Fixed issue with chosen version cookie.

==

Sending        application/controllers/auth.php
Sending        application/libraries/MY_Controller.php
Transmitting file data ..
Committed revision 2675.

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; thanks, Ryan.