If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Not able to add selected add-ons to facebook page

VERIFIED FIXED in 3.4.7

Status

addons.mozilla.org Graveyard
Facebook Application
--
critical
VERIFIED FIXED
9 years ago
2 years ago

People

(Reporter: SthrnDixieCwgrl, Assigned: fligtar)

Tracking

unspecified
3.4.7

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9) Gecko/2008052906 Firefox/3.0
Build Identifier: 

I went to my Facebook profile and clicked on the Rock your Firefox application.  Then I went under Import Add-ons and clicked on Find out more.  Then I clicked on Detect My Add-ons.  Then I followed the instructions on the screen.  Then I clicked continue.  Then I clicked on Add Selected to Favorites.  Then I got the following error message:

There are errors in this form. Please correct them and resubmit.


Reproducible: Always

Steps to Reproduce:
1. Logged into Facebook
2. Clicked on the Rock your Firefox application
3. Clicked on Find out more under Import Add-ons
4. Clicked on Detect My Add-ons
5. Followed the instructions on the screen
6. Then I clicked continue
7. Clicked on Add Selected to Favorites
Actual Results:  
Received following error message:

There are errors in this form. Please correct them and resubmit.

Expected Results:  
Add the detected Add-ons to My Favorite Add-ons section of my profile
(Reporter)

Updated

9 years ago
Severity: major → critical
Confirmed using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9.0.1) Gecko/2008070206 Firefox/3.0.1; I'll attach a log showing:

* update ping (add-on scan)
* successful ping
* failed submit
Status: UNCONFIRMED → NEW
Ever confirmed: true
Created attachment 333701 [details]
Logfile (HTML)

Could this be some CSRF change fallout?
Oh; looks like it's failing to find https://addons.mozilla.org/en-US/firefox/js/__utm.js (404).  Isn't that just the Urchin tracker, though?
(Assignee)

Updated

9 years ago
Assignee: nobody → fligtar
OS: Windows Vista → All
Hardware: PC → All
Target Milestone: --- → 3.4.7
(Assignee)

Comment 4

9 years ago
Created attachment 333982 [details] [diff] [review]
patch

This was caused by the CSRF check. We haven't updated RYF specifically to avoid these issues, but this part of the app runs on regular AMO so it was affected. This patch just adds the action to the CSRF exception list.

Fred, if you want to test the actual patch, you'll need to pull a session id from facebook_sessions in your db and insert it into:
http://your-amoinstall/en-US/firefox/facebookinstall/import/{sessionid}/results

Before the patch, if you submit that page's form, you'll get an error. After, you should be redirected to facebook.
Attachment #333982 - Flags: review?(fwenzel)
(Reporter)

Comment 5

9 years ago
I clicked on http://your-amoinstall/en-US/firefox/facebookinstall/import/{sessionid}/results

Address Not Found

      

      
      
      

      
        
        

          

Firefox can't find the server at www.your-amoinstall.com.

        


        
        

The browser could not find the host server for the provided address.

    * Did you make a mistake when typing the domain? (e.g. "ww.mozilla.org" instead of "www.mozilla.org")
    * Are you certain this domain address exists?  Its registration may have expired.
    * Are you unable to browse other sites?  Check your network connection and DNS server settings.
    * Is your computer or network protected by a firewall or proxy?  Incorrect settings can interfere with Web browsing.

How do I fix this

(In reply to comment #4)
> Created an attachment (id=333982) [details]
> patch
> 
> This was caused by the CSRF check. We haven't updated RYF specifically to avoid
> these issues, but this part of the app runs on regular AMO so it was affected.
> This patch just adds the action to the CSRF exception list.
> 
> Fred, if you want to test the actual patch, you'll need to pull a session id
> from facebook_sessions in your db and insert it into:
> http://your-amoinstall/en-US/firefox/facebookinstall/import/{sessionid}/results
> 
> Before the patch, if you submit that page's form, you'll get an error. After,
> you should be redirected to facebook.
> 

Comment on attachment 333982 [details] [diff] [review]
patch

I checked out the patch and it works as described. If you don't select any add-ons from the list though (or all are your favorites already), you get a bunch of warnings due to the empty list. You may want to fix that, but it's a different bug.
Attachment #333982 - Flags: review?(fwenzel) → review+
(In reply to comment #5)
> I clicked on
> http://your-amoinstall/en-US/firefox/facebookinstall/import/{sessionid}/results
> 
> Address Not Found

The instructions were for me to check out Justin's patch only. If you have additional questions about addons.mozilla.org, please feel free to email amo-editors@mozilla.org or file a new bug here in bugzilla.
(Assignee)

Comment 8

9 years ago
Checked into r17697. Thanks!
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Keywords: push-needed
Resolution: --- → FIXED

Updated

9 years ago
Keywords: push-needed
Wait a minute, I don't know if the facebook part was pushed public yet. Fligtar: If it was published, can you please remove the push-needed keyword? Thanks!
Keywords: push-needed
(Assignee)

Comment 10

9 years ago
This part of the app is hosted on normal AMO, so it was pushed.
Keywords: push-needed
Verified FIXED; https://addons.mozilla.org/en-US/firefox/facebookinstall/import/6b289e40b5cff7409a81592cb3d92b65986288/results# returned me to http://apps.new.facebook.com/add-ons/favorites/?added=201%2C3829.
Status: RESOLVED → VERIFIED
(Reporter)

Comment 12

9 years ago
I went to Facebook and I tried this add-on again.  It worked. Thank you.

(In reply to comment #11)
> Verified FIXED;
> https://addons.mozilla.org/en-US/firefox/facebookinstall/import/6b289e40b5cff7409a81592cb3d92b65986288/results#
> returned me to
> http://apps.new.facebook.com/add-ons/favorites/?added=201%2C3829.
> 

Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.