Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible bug in Meteor.http #553

Closed
vlebedev opened this issue Dec 15, 2012 · 4 comments
Closed

Possible bug in Meteor.http #553

vlebedev opened this issue Dec 15, 2012 · 4 comments
Labels

Comments

@vlebedev
Copy link

@vlebedev vlebedev commented Dec 15, 2012

I'm trying to get a list of facebook friends for a current user (client-side):

Meteor.http.get "https://graph.facebook.com/#{fid}/friends?access_token=#{access_token}",
    (error, result) ->
        if result.statusCode is 200
            if result.data
                do_something result
            else
                console.log "Error: 'data' property is empty in facebook friends request"
        else
            console.log "Error: facebook friends request returned #{result.statusCode}"

The code above works well in Chrome (23.0), but it fails in Firefox (17.0.1) - result.data is always null despite correct data is presented in result.content property (in a string format, of course).

Looks like this is a browser-dependent bug in Meteor.http.

@droizman
Copy link

@droizman droizman commented Dec 19, 2012

I'm having the same problem. Fine on Chrome, null on Firefox.

@n1mmy
Copy link
Member

@n1mmy n1mmy commented Jan 9, 2013

Replicated and confirmed. Will investigate further.

@n1mmy
Copy link
Member

@n1mmy n1mmy commented Jan 12, 2013

Thanks for the report, @vlebedev, and thanks for including enough info to replicate easily.

This appears to be a bug in Firefox. Possibly this one: https://bugzilla.mozilla.org/show_bug.cgi?id=608735

I've committed a workaround to the devel branch in 448da70. It seems to work for me, also talking to Facebook.

Let me know if this works for you.

@n1mmy n1mmy closed this Jan 12, 2013
@vlebedev
Copy link
Author

@vlebedev vlebedev commented Jan 13, 2013

Thanks a lot @n1mmy! Will check and let you know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants