Skip to content

feat: support OIDC endpoints - #1630

Merged
jacalata merged 2 commits into
tableau:developmentfrom
jorwoods:jorwoods/oidc
Aug 1, 2025
Merged

feat: support OIDC endpoints#1630
jacalata merged 2 commits into
tableau:developmentfrom
jorwoods:jorwoods/oidc

Conversation

@jorwoods

@jorwoods jorwoods commented Jul 3, 2025

Copy link
Copy Markdown
Contributor

Add support for remaining OIDC endpoints, including getting an OIDC configuration by ID, removing the configuration, creating, and updating configurations.

@jorwoods
jorwoods force-pushed the jorwoods/oidc branch 4 times, most recently from fb34e81 to 825e6cc Compare July 3, 2025 14:05
Add support for remaining OIDC endpoints, including getting an
OIDC configuration by ID, removing the configuration, creating,
and updating configurations.
@jacalata
jacalata merged commit 61062dc into tableau:development Aug 1, 2025
22 checks passed
@jorwoods
jorwoods deleted the jorwoods/oidc branch August 1, 2025 11:24
jacalata added a commit that referenced this pull request Feb 2, 2026
* feat: New APIs: Update multiple connections in a single workbook/datasource (#1638)
* feat: enable toggling attribute capture for a site (#1619)
* feat: support OIDC endpoints (#1630)
* feat: add WebAuthoringForFlows capability to Permission class (#1642) 
* feat: support collections in favorites (#1647)
* feat: Add UAT (unified access token) support to JWT login (#1671)
* feat: Update permissions_item.py --added ExtractRefresh attribute (#1617) (#1669)
* feat: make refresh consistent between endpoints (#1665)
* feat: make ResourceReference hashable (#1668) Closes #1666
* feat: delete view (#1712)
* feat: batch create schedule (#1714)
* feat: users csv import (#1409)
* feat: implement users bulk_remove
* feat: enable idp_configuration_id in bulk_add
* feat: support extensions api (#1672)
* feat: Add support for receiving "Customized Monthly" schedule intervals (#1670)
* feat: implement #816: project.get_by_id (#1736)

* fix: put special fields first (#1622) Closes #1620
* fix: virtual connections username (#1628) Closes #1626
* fix: add contentType to tags batch actions (#1643)
* fix: datasource owner/project missing parsing (#1700)
* fix: datasource description update and publish (#1682)
* fix: windows decoding error
* fix: assert on warning instead of ignore
* fix: add workbook and view setter for custom view (#1730)
* fix: handle parameters for view filters (#1633)
* fix: handle parameters for view filters Closes #1632
jacalata pushed a commit that referenced this pull request Feb 3, 2026
* feat: support OIDC endpoints

Add support for remaining OIDC endpoints, including getting an
OIDC configuration by ID, removing the configuration, creating,
and updating configurations.

* feat: add str and repr to oidc item

---------

Co-authored-by: Jordan Woods <13803242+jorwoods@users.noreply.github.com>
jacalata pushed a commit that referenced this pull request Feb 3, 2026
* feat: support OIDC endpoints

Add support for remaining OIDC endpoints, including getting an
OIDC configuration by ID, removing the configuration, creating,
and updating configurations.

* feat: add str and repr to oidc item

---------

Co-authored-by: Jordan Woods <13803242+jorwoods@users.noreply.github.com>
vchavatapalli pushed a commit to vchavatapalli/server-client-python that referenced this pull request May 5, 2026
* feat: New APIs: Update multiple connections in a single workbook/datasource (tableau#1638)
* feat: enable toggling attribute capture for a site (tableau#1619)
* feat: support OIDC endpoints (tableau#1630)
* feat: add WebAuthoringForFlows capability to Permission class (tableau#1642) 
* feat: support collections in favorites (tableau#1647)
* feat: Add UAT (unified access token) support to JWT login (tableau#1671)
* feat: Update permissions_item.py --added ExtractRefresh attribute (tableau#1617) (tableau#1669)
* feat: make refresh consistent between endpoints (tableau#1665)
* feat: make ResourceReference hashable (tableau#1668) Closes tableau#1666
* feat: delete view (tableau#1712)
* feat: batch create schedule (tableau#1714)
* feat: users csv import (tableau#1409)
* feat: implement users bulk_remove
* feat: enable idp_configuration_id in bulk_add
* feat: support extensions api (tableau#1672)
* feat: Add support for receiving "Customized Monthly" schedule intervals (tableau#1670)
* feat: implement tableau#816: project.get_by_id (tableau#1736)

* fix: put special fields first (tableau#1622) Closes tableau#1620
* fix: virtual connections username (tableau#1628) Closes tableau#1626
* fix: add contentType to tags batch actions (tableau#1643)
* fix: datasource owner/project missing parsing (tableau#1700)
* fix: datasource description update and publish (tableau#1682)
* fix: windows decoding error
* fix: assert on warning instead of ignore
* fix: add workbook and view setter for custom view (tableau#1730)
* fix: handle parameters for view filters (tableau#1633)
* fix: handle parameters for view filters Closes tableau#1632
jacalata added a commit that referenced this pull request Jul 29, 2026
…ants

Two adversarial-review cleanups on tableauserverclient/types.py:

- HasIdpConfigurationID was `str | IDPAttributes`, and every caller
  annotation used `str | HasIdpConfigurationID`, which expanded to the
  redundant `str | str | IDPAttributes`. Narrowed the alias to a pure
  Protocol union so callers can write `str | HasIdpConfigurationID`
  without stuttering.

- IDPAttributes covers objects that declare `idp_configuration_id` as a
  plain class attribute; IDPProperty covers objects that declare it as
  a read-only @Property. Both were introduced in #1630 but only
  IDPAttributes was ever in HasIdpConfigurationID's union, so the
  @Property variant would fail mypy against the endpoint's public
  signature. HasIdpConfigurationID now unions both to match the
  original design intent.

- Added a docstring block explaining why both Protocol variants exist.

- Cleaned up unused IDPAttributes/IDPProperty imports in oidc_endpoint.py
  (only HasIdpConfigurationID is referenced there).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

2 participants