You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Add support for fine grained personal access tokens to minimize the permissions needed to retrieve data from tables of interest to the user (up to the user to ensure they have added the right permissions to the FG PAT).
Describe the solution you'd like
This is a simple change in 4 places in connect and connectV4. Everywhere you validate that the supplied token has the ghp_ prefix, also allow the github_pat_ prefix. I tested that this works as expected.
The text was updated successfully, but these errors were encountered:
@graza-io, @cbruno10, Further more — I have a few findings related to the fine-grained access token.
The error Resource not accessible by personal access token is caused by a GraphQL field error. You can refer to this documentation on how to handle such errors.
I tested the fix with the github_issue and github_my_issue tables, and pushed some changes to the branch add-support-for-fine-grained-access-token. The code changes seem to be working fine.
However, I’m not entirely sure if this is the best solution. Any thoughts would be appreciated.
Hello everyone, I support Fine-grained personal access token (github_pat_)
In large enterprise companies, the security department may refuse to use Personal access token (classic).
Is your feature request related to a problem? Please describe.
Add support for fine grained personal access tokens to minimize the permissions needed to retrieve data from tables of interest to the user (up to the user to ensure they have added the right permissions to the FG PAT).
Describe the solution you'd like
This is a simple change in 4 places in
connect
andconnectV4
. Everywhere you validate that the supplied token has theghp_
prefix, also allow thegithub_pat_
prefix. I tested that this works as expected.The text was updated successfully, but these errors were encountered: