Skip to content

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

@MarkusH 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

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

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

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

MarkusH commented Feb 29, 2012

Copy link
Copy Markdown
Author

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

@stackmagic

Copy link
Copy Markdown

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

@mvlcek

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 fernandezpablo85 merged commit 19b6f18 into scribejava:master Jul 28, 2012
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