dbox icon indicating copy to clipboard operation
dbox copied to clipboard

Oauth2 and accesstoken

Open CrackerakiUA opened this issue 8 years ago • 6 comments

I am using this strategy from passport LINK From that strategy i am getting access_token from dropbox, but when i put that access_token inside var client = app.client(access_token) it throws me an message { error_description: 'No auth function available for given request', error: 'invalid_request' } and status 400.

CrackerakiUA avatar Apr 27 '16 21:04 CrackerakiUA

g00dv1n avatar May 06 '16 10:05 g00dv1n

this modules looks pretty outdated

CrackerakiUA avatar May 06 '16 10:05 CrackerakiUA

I don’t see what disparaging this library accomplishes.

This is a JavaScript library that wraps HTTP calls to a REST API. There is nothing special going on here. Before you throw the baby out with the bath water consider the following.

  • dbox has been used to sync terabytes of data in production environment.
  • dbox has 25 releases - all with release notes and tests.
  • dbox makes the OAuth handshake suck way less.

If you have found a bug and it would help your life if it were fixed the best thing you can do for yourself is fix it. I have found the best way to do this is by writing a failing test that expects the correct behavior, then changing the implementation to make the test pass. This will also help maintainers evaluate the PR mainly because it will demonstrate that the other tests do not fail.

If you are unwilling to fix but still want to be a good open source citizen then it is appreciated if you open an issue explaining the expected behavior and your findings. Include any other useful information when you do this such as version of node (super important these days) and operating system (things with file paths can differ greatly from system to system).

If you can’t fix it but want it fixed, then you are in the absolute best condition to learn something new. Ask a friend you know for help.

If Dropbox has totally deprecated this API then we probably need someone to look into the details and see if this library is work keeping active our not. Probably something anyone can do. I stopped paying attention to what Dropbox is up to so I am out of the loop.

Although I still use it, this library has for the most part outlived its usefulness to me so its really up to the community to pick up the torch if its wanted.

cheers, Brock

sintaxi avatar May 06 '16 12:05 sintaxi

@sintaxi for old projects where this were using it's will be good to play with it. But for new projects there are v2 of dropbox api and passport authentication which is easier. Bad thing here is that in readme we don't say something that we use v1, i spend some time to found it.

CrackerakiUA avatar May 07 '16 16:05 CrackerakiUA

@CrackerakiUA Then help the hole community and make a pull request with a change to the readme stating that this only works for v1 (and a link to another repo if a person would wanna use v2, if you found another repo).

jstroem avatar May 09 '16 06:05 jstroem

@sintaxi i have started other plugin with v2, but it's still under developing.

CrackerakiUA avatar May 09 '16 14:05 CrackerakiUA