Skip to content

Conversation

@jorwoods
Copy link
Contributor

Add JWTAuth class, as well as allowing Token auth to do impersonation.

@jacalata jacalata requested a review from LGraber April 13, 2023 07:21
Copy link
Contributor

@jacalata jacalata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you just add a repr function for the JWTAuth class? something like
return "".format(self.jwt[:5]+ "...", self.site_id)

@jorwoods
Copy link
Contributor Author

Done. How do you feel about explicitly marking the Credentials.__repr__ and Credentials.credentials as abstractmethods?

Additionally, I feel like sign_in_with_personal_access_token should be deprecated since it is just a redundant passthrough.

@jorwoods
Copy link
Contributor Author

Was there something missing from this PR that I should have implemented?

@jacalata
Copy link
Contributor

oops, I think this got auto-closed when I merged the dev branch

@jacalata jacalata reopened this Apr 25, 2023
@ncattelainbetclic
Copy link

Will this branch be merged soon? Thanks in advance

@ngohuuduc
Copy link

Hello,
Is there any update regarding to this matter ?

@jacalata jacalata merged commit f56b2c7 into tableau:development Aug 1, 2023
@jorwoods jorwoods deleted the jorwoods/JWTAuth branch June 7, 2024 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants