Skip to content

fix behavior when url has no protocol - #1125

Merged
jacalata merged 2 commits into
developmentfrom
jac/url-recognition
Oct 6, 2022
Merged

fix behavior when url has no protocol#1125
jacalata merged 2 commits into
developmentfrom
jac/url-recognition

Conversation

@jacalata

Copy link
Copy Markdown
Contributor

Defaulting to http if no protocol is entered, since that is the behavior we expect in tabcmd currently.
Manually handle a redirect on login, so we don't send a GET request that fails.

@bhuvneshdevbhuvneshdev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Comment threadtableauserverclient/server/endpoint/auth_endpoint.py
Comment threadtableauserverclient/server/endpoint/auth_endpoint.py
Comment threadtableauserverclient/server/server.py
Base automatically changed from development to masterSeptember 28, 2022 07:41
Comment threadtableauserverclient/models/server_info_item.py
t8y8
t8y8 approved these changes Oct 5, 2022
@jacalata
jacalata changed the base branch from master to developmentOctober 6, 2022 17:41
@jacalata
jacalata merged commit ca4d79e into developmentOct 6, 2022
@jacalata
jacalata deleted the jac/url-recognition branch October 6, 2022 17:57
jacalata added a commit that referenced this pull request Oct 11, 2022
* Allow injection of session_factory to allow use of a custom session
* Jac/show server info (#1118)
* Fix bug in exposing ExcelRequestOptions and test (#1123)
* Fix a few pylint errors (#1124)
* fix behavior when url has no protocol (#1125)
* smoke test for pypi
* Add permission control for Data Roles and Metrics (Issue #1063) (#1120)
Co-authored-by: Marwan Baghdad <mrwanbaghdad76@gmail.com>
Co-authored-by: jorwoods <jorwoods@users.noreply.github.com>
Co-authored-by: Brian Cantoni <bcantoni@salesforce.com>
Co-authored-by: TrimPeachu <77048868+TrimPeachu@users.noreply.github.com>
jacalata added a commit that referenced this pull request Feb 14, 2023
* Allow injection of sessions (#1111)
* show server info (#1118)
* Fix bug in exposing ExcelRequestOptions and test (#1123)
* Fix a few pylint errors (#1124)
* fix behavior when url has no protocol (#1125)
* Add permission control for Data Roles and Metrics (Issue #1063) (#1120)
* add option to pass specific datasources (#1150)
* allow user agent to be set by caller (#1166)
* Fix issues with connections publishing workbooks (#1171)
* Allow download to file-like objects (#1172)
* Add updated_at to JobItem class (#1182)
* fix revision references where xml returned does not match docs (#1176)
* Do not create empty connections list (#1178)
---------
Co-authored-by: Marwan Baghdad <mrwanbaghdad76@gmail.com>
Co-authored-by: jorwoods <jorwoods@users.noreply.github.com>
Co-authored-by: Brian Cantoni <bcantoni@salesforce.com>
Co-authored-by: TrimPeachu <77048868+TrimPeachu@users.noreply.github.com>
Co-authored-by: Stu Tomlinson <stu@nosnilmot.com>
Co-authored-by: Jeremy Harris <jercharris89@gmail.com>
vchavatapalli pushed a commit to vchavatapalli/server-client-python that referenced this pull request May 5, 2026
* Allow injection of session_factory to allow use of a custom session
* Jac/show server info (tableau#1118)
* Fix bug in exposing ExcelRequestOptions and test (tableau#1123)
* Fix a few pylint errors (tableau#1124)
* fix behavior when url has no protocol (tableau#1125)
* smoke test for pypi
* Add permission control for Data Roles and Metrics (Issue tableau#1063) (tableau#1120)
Co-authored-by: Marwan Baghdad <mrwanbaghdad76@gmail.com>
Co-authored-by: jorwoods <jorwoods@users.noreply.github.com>
Co-authored-by: Brian Cantoni <bcantoni@salesforce.com>
Co-authored-by: TrimPeachu <77048868+TrimPeachu@users.noreply.github.com>
vchavatapalli pushed a commit to vchavatapalli/server-client-python that referenced this pull request May 5, 2026
* Allow injection of sessions (tableau#1111)
* show server info (tableau#1118)
* Fix bug in exposing ExcelRequestOptions and test (tableau#1123)
* Fix a few pylint errors (tableau#1124)
* fix behavior when url has no protocol (tableau#1125)
* Add permission control for Data Roles and Metrics (Issue tableau#1063) (tableau#1120)
* add option to pass specific datasources (tableau#1150)
* allow user agent to be set by caller (tableau#1166)
* Fix issues with connections publishing workbooks (tableau#1171)
* Allow download to file-like objects (tableau#1172)
* Add updated_at to JobItem class (tableau#1182)
* fix revision references where xml returned does not match docs (tableau#1176)
* Do not create empty connections list (tableau#1178)
---------
Co-authored-by: Marwan Baghdad <mrwanbaghdad76@gmail.com>
Co-authored-by: jorwoods <jorwoods@users.noreply.github.com>
Co-authored-by: Brian Cantoni <bcantoni@salesforce.com>
Co-authored-by: TrimPeachu <77048868+TrimPeachu@users.noreply.github.com>
Co-authored-by: Stu Tomlinson <stu@nosnilmot.com>
Co-authored-by: Jeremy Harris <jercharris89@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@jacalata@t8y8@bhuvneshdev@anyoung-tableau