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

Authorize URL should not be under api.fitbit.com #38

Closed
ElMassimo opened this issue May 19, 2015 · 4 comments
Closed

Authorize URL should not be under api.fitbit.com #38

ElMassimo opened this issue May 19, 2015 · 4 comments

Comments

@ElMassimo
Copy link

According to the Fitbit documentation, in order for the users to authorize the client application, users should be redirected to:
https://www.fitbit.com/oauth/authorize?oauth_token=[&locale=][&display=touch][&requestCredentials=true]

With the current configuration, it seems like users are being redirected to https://api.fitbit.com/oauth/authorize.

According to fitbit support, there are two reasons to use the non-api URL:

  1. When you send people to api.fitbit.com, they are always required to sign in to authorize your app. When your app uses the correct URL, people do not have to sign in if they are already signed in on the device.
  2. While Fitbit has supported authorization on api.fitbit.com, soon it will begin redirecting these requests to www.fitbit.com, which will increase the page load time for your users.
@ghost
Copy link

ghost commented May 19, 2015

Hello, I'm the API lead at Fitbit. I can confirm that the OAuth 1.0a authorization URL should use www.fitbit.com. I know many developers have used Fitgem and this change would be valuable to them for the reasons above.

@whazzmaster
Copy link
Owner

Apologies- work has been hectic lately. I'll get PR #39 merged and see if I can cut a new gem version tonight.

@whazzmaster
Copy link
Owner

One question @jeremiahlee; so all API requests continue to hit api.fitbit.com and it's just the OAuth authorize call that needs to go to www.fitbit.com?

Damn my laziness- the docs said it plainly: authorize only hits www.fitbit.com. Ok.

@whazzmaster
Copy link
Owner

Closed by #39

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

No branches or pull requests

2 participants