node-harvest icon indicating copy to clipboard operation
node-harvest copied to clipboard

Add in OAuth functionality.

Open simplyspoke opened this issue 7 years ago • 4 comments

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

simplyspoke avatar Apr 09 '18 03:04 simplyspoke

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.

BramEsposito avatar Jan 28 '20 16:01 BramEsposito

Will do and thanks for the notice.

simplyspoke avatar Jan 28 '20 19:01 simplyspoke

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 [email protected] 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 https://github.com/simplyspoke/node-harvest/issues/85?email_source=notifications&email_token=ABOTPLKGCHLH7BSCSSIEGA3RE7TURA5CNFSM4EZPTBR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENFAPQQ#issuecomment-592054210, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABOTPLK6ZGM4QA2FDGV7OELRE7TURANCNFSM4EZPTBRQ .

simplyspoke avatar Feb 28 '20 01:02 simplyspoke

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

Pezmc avatar Jun 22 '20 14:06 Pezmc