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

Use access token for all requests if available #191

Closed
wants to merge 1 commit into from

Conversation

ScottPolhemus
Copy link

Per the recent platform changes (#182), all requests should include an access token. This change uses the token for all API calls if it's been provided, preserving support for apps with legacy client IDs that don't yet require auth for certain methods.

@foxshark
Copy link

@ScottPolhemus - this was the key to making public requests work after the change #182 as you pointed out. I needed to add the public_content scope to make the token generation work fully, otherwise I had the following error:

"error_type":"OAuthPermissionsException","code":400,"error_message":"This request requires scope=public_content, but this access token is not authorized with this scope. The user must re-authorize your application with scope=public_content to be granted this permissions."

Please accept my pull request to include this to your pull request.

@TiS
Copy link

TiS commented Jun 7, 2016

Can you PLEASE merge this PR? This is really essential to work with current Instagram API.

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

Successfully merging this pull request may close these issues.

3 participants