8000 UserItem class auth_setting needs updating · Issue #1216 · tableau/server-client-python · GitHub
[go: up one dir, main page]
Skip to content

UserItem class auth_setting needs updating #1216

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.

Sign up for GitHub

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

Jump to bottom
Closed
ma7tcsp opened this issue Apr 12, 2023 · 0 comments
Closed

UserItem class auth_setting needs updating #1216

ma7tcsp opened this issue Apr 12, 2023 · 0 comments
Labels
0.26 fixed gap A gap in the library for existing server functionality

Comments

@ma7tcsp
Copy link
Contributor
ma7tcsp commented Apr 12, 2023 •
edited
Loading

In TSC this parameter only accepts "ServerDefault" or "SAML".

However, in Tableau Cloud where MFA has been turned on (ie it now HAS to be turned on) this is not enough.

TSC ServerDefault maps to REST API TableauID => TableauID is no longer a valid auth
REST API provides TableauIDWithMFA, and also OpenID.

OpenID is actually in the user_item.py model, however not updated in the documentation.

We need a way to set this additional REST API auth-setting values as TSC auth_setting values too. Otherwise we can't use TSC to add users to Tableau Cloud anymore.
ma7tcsp added a commit to ma7tcsp/server-client-python that referenced this issue Apr 12, 2023
@ma7tcsp
Update user_item.py
af820ce 
TableauIDWithMFA added to the user_item model to allow creating users on Tableau Cloud with MFA enabled and to keep in sync with REST API.
Fixing Issue tableau#1216
@ma7tcsp ma7tcsp mentioned this issue Apr 12, 2023
Merged
jacalata pushed a commit that referenced this issue Apr 12, 2023
@ma7tcsp
Update user_item.py (#1217)
fc9568d 
TableauIDWithMFA added to the user_item model to allow creating users on Tableau Cloud with MFA enabled and to keep in sync with REST API.
Fixing Issue #1216
@jacalata jacalata mentioned this issue Apr 12, 2023
Merged
@jacalata jacalata added fixed gap A gap in the library for existing server functionality 0.26 labels Apr 19, 2023
jacalata added a commit that referenced this issue Sep 22, 2023
@jacalata @ma7tcsp
@LarsBreddemann @jorwoods @syou6162 @bcantoni @a-torres-2 @wlodi83
0.27 (#1272)
341dcd2 
* fix: TableauIDWithMFA added to the user_item model to allow creating users on Tableau Cloud with MFA enabled (#1216)
* fix: make project optional in datasources #1210
* fix: allow setting timeout on workbook endpoint #1087
* fix: can't certify datasource on publish #1058 
* fix filter in operator spaces bug (#1259)
* fix: remove logging configuration from TSC (#1248)
* Hotfix schedule_item.py for issue 1237 (#1239), Remove duplicate assignments to fields (#1244)
* Fix shared attribute for custom views (#1280)

New functionality
* enable filtering for Excel downloads #1209, #1281
* query view by content url #456  
* update datasource to use bridge (#1224)
* Add JWTAuth, add repr using qualname
* Add publish samples attribute (#1264)
* add support for custom schedules in TOL (#1273)
* Enable asJob for group update (#1276)


Co-authored-by: Tim Payne <47423639+ma7tcsp@users.noreply.github.com>
Co-authored-by: Lars Breddemann <139097050+LarsBreddemann@users.noreply.github.com>
Co-authored-by: jorwoods <jorwoods@users.noreply.github.com>
Co-authored-by: Austin <110413815+austinpeters-gohealthuccom@users.noreply.github.com>
Co-authored-by: Yasuhisa Yoshida <syou6162@gmail.com>
Co-authored-by: Brian Cantoni <bcantoni@salesforce.com>
Co-authored-by: a-torres-2 <142839181+a-torres-2@users.noreply.github.com>
Co-authored-by: Łukasz Włodarczyk <lukasz@pushmetrics.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0.26 fixed gap A gap in the library for existing server functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jacalata @ma7tcsp
0