FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update the Dropbox API to use correct URLs by MarkusH · Pull Request #203 · scribejava/scribejava · GitHub

Update the Dropbox API to use correct URLs - #203

Merged
fernandezpablo85 merged 1 commit into
scribejava:masterfrom
MarkusH:dropbox
Jul 28, 2012
Merged

Update the Dropbox API to use correct URLs#203
fernandezpablo85 merged 1 commit into
scribejava:masterfrom
MarkusH:dropbox

Conversation

MarkusH commented Dec 13, 2011

Copy link
Copy Markdown

The Dropbox OAuth API uses new URLs for the request-token and authorize-token process. See https://www.dropbox.com/developers/reference/api#request-token

This commit changes the URLs to the correct endpoints.

sigbjod commented Jan 20, 2012

Copy link
Copy Markdown

Great that you've provided a simple update regarding the changes in the Dropbox API! Have you considered supporting the optional oauth_callback and locale parameters in the authorize step?

See https://www.dropbox.com/developers/reference/api#authorize for more information regarding these parameters.

MarkusH commented Jan 20, 2012

Copy link
Copy Markdown
Author

No, I have not considered that feature since that would be an enhancement and no fix.

sigbjod commented Jan 20, 2012

Copy link
Copy Markdown

I can't say I disagree with that approach.

As you might have guessed, I'd like to pass the callback parameter and I'll see if I'm able to find a simple and clean solution to this enhancement.

MarkusH commented Feb 29, 2012

Copy link
Copy Markdown
Author

So. After a month of inactivity in this pull request: will it be merged?

Copy link
Copy Markdown

merged into my own fork, i intend to provide a release in my maven repo later today

mvlcek commented Jul 26, 2012

Copy link
Copy Markdown

I've added the callback functionality (and an additional OAuth 1.0 service/API) to my fork, see #287

fernandezpablo85 added a commit that referenced this pull request Jul 28, 2012
Update the Dropbox API to use correct URLs
fernandezpablo85 merged commit 19b6f18 into scribejava:master Jul 28, 2012
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

5 participants


Back | FazBrowse Home | New Git URL