-
Notifications
You must be signed in to change notification settings - Fork 448
Update JWT Login Docs to include Unified Access Token, and and fix PAT table formatting #1718
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
base: gh-pages
Are you sure you want to change the base?
Conversation
|
Thanks for the contribution! It looks like @ruby-sf is an internal user so signing the CLA is not required. However, we need to confirm this. |
|
Close and reopen to see if it fixes the CLA issue. |
|
The plan is for @jacalata to merge this when we publish the next release. |
bcantoni
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rubywerman I added a couple more suggested improvement here
|
|
||
| ```py | ||
| class JWTAuth(Credentials): | ||
| def __init__(self, jwt=None, site_id=None, user_id_to_impersonate=None): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add isUat here as well
| import tableauserverclient as TSC | ||
|
|
||
| # Connected App JWT | ||
| tableau_auth = TSC.JWTAuth('JWT', 'SITENAME') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest making both of these examples more clear by providing the argument name as well (better practice), jwt='JWT', etc
In December 2025, we added UAT, a new JWT-based auth type for Tableau Cloud: https://help.tableau.com/current/api/cloud-manager/en-us/docs/unified_access_tokens.html
@BereketBirbo added UAT to the TSC library here: https://github.com/tableau/server-client-python/pulls?q=is%3Apr+author%3ABereketBirbo+is%3Aclosed., which has been merged into the development branch.
This MR updates the docs to show the new parameter, and fixed a formatting issue on the PAT section (the table wasn't showing up)



Before:
After:
JWT updates: