-
Notifications
You must be signed in to change notification settings - Fork 57
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
Add in OAuth functionality. #85
Comments
I built a module for authenticating on Harvest with Express and Grant: https://github.com/BramEsposito/node-harvest-auth/ Check an example implementation here It can be extended to use Koa or Hapi. |
Will do and thanks for the notice. |
Awesome. I think I have a little breather here to review. Mind if I pull it
in if it looks good?
…On Thu, Feb 27, 2020, 10:35 AM Pez Cuckow ***@***.***> wrote:
I have a branch where I added support for this back a while back!
https://github.com/Pezmc/node-harvest
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#85?email_source=notifications&email_token=ABOTPLKGCHLH7BSCSSIEGA3RE7TURA5CNFSM4EZPTBR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENFAPQQ#issuecomment-592054210>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABOTPLK6ZGM4QA2FDGV7OELRE7TURANCNFSM4EZPTBRQ>
.
|
I have a very old fork of this project that implements oauth and image uploads: https://github.com/Pezmc/node-harvest/tree/custom-build We could likely rebase those changes onto your current master to add support? But it looks like lots of changes have been made to node-harvest since then? I opened a draft PR so you can visualize the changes: https://github.com/simplyspoke/node-harvest/pull/451/files |
With the release of v2, the older OAuth methods are no longer viable. Effort is needed to create a new method compatible with this API version
The text was updated successfully, but these errors were encountered: