search

singer-io / tap-harvest

A Singer tap for extracting data from the Harvest API
GNU Affero General Public License v3.0
8 stars 13 forks source link

Authentication with Personal Access Token #30

Open jack89roberts opened 5 years ago

jack89roberts commented 5 years ago

First of all thanks for the work put into this package, we're keen to give it a try in our team.

At the moment it seems the config file/authentication is expecting OAuth2. Is it possible to set it up with a Personal Access Token instead? If not would this be straightforward for me to add with a few alterations to init.py or is it more complicated/is there a good reason to not use this with Personal Access Tokens?

Thanks!

cdilga commented 5 years ago

@jack89roberts , I found the same issue that you had with the tap-harvest-forecast, which is another product by harvest and uses the same auth flow. Check out the updates to the documentation I made over on a fork

jack89roberts commented 5 years ago

@cdilga Nice, thanks! I'm not actively working on this anymore but if/when I get back to it I'll be sure to check it out.

tauno commented 5 years ago

I started a fork that adds support for personal tokens - https://github.com/thinkshout/tap-harvest/tree/personal-token. Needs documentation updates and ideally a check that either the oauth settings or the personal token settings are there.