chore: remove legacy REST API version shim for servers pre-2.4
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 55/100
Direção de pesquisa
Review server.py, especially _PRODUCT_TO_REST_VERSION, _get_legacy_version(), and _determine_highest_version(). First resolve whether the minimum supported version should be 2.4 or 3.0, then remove the legacy compatibility paths and update the minimum and default versions so pre-2.4 servers are no longer supported.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Summary
Remove the compatibility shim that supports connecting to Tableau Server versions that predate REST API 2.4 (i.e., Tableau Server 10.0 and earlier, released before 2017).
Background
server.py currently has:
_PRODUCT_TO_REST_VERSION = {
"10.0": "2.3",
"9.3": "2.2",
...
}
minimum_supported_server_version = "2.3"
default_server_version = "2.4" # first version that dropped the legacy auth endpoint
The _get_legacy_version() method hits /auth?format=xml — a pre-2.4 endpoint — because servers that old don't support the standard /serverinfo REST API endpoint. _determine_highest_version() falls back to this when serverInfo returns a 404.
What to remove
_PRODUCT_TO_REST_VERSIONdict_get_legacy_version()method- The two
exceptbranches in_determine_highest_version()that call_get_legacy_version() - Raise
minimum_supported_server_versionto"2.4"(or"3.0"— see below) - Update
default_server_versionto match
Impact on users
None, unless they are connecting to Tableau Server 10.0 (released 2016) or earlier. Tableau Server 10.x reached end of life in 2019. All currently-supported Tableau Server versions use REST API 3.x.
Note: the @api(version="2.x") decorators on endpoint methods are minimum version requirements, not upper bounds — they are unaffected by this change and no endpoint methods are removed.
Decision needed
Should minimum_supported_server_version be raised to "2.4" (first version with /serverinfo) or "3.0" (Tableau Server 2018.1, 6+ years old)? The code change is identical either way; the difference is only the documented policy.
- Linguagem predominante
- Python
- Estrelas
- 716
- Forks
- 446
- Merge médio
- 8d 8h
- PRs com merge (30d)
- 2
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de tableau/server-client-python
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
tableau/server-client-python#1865 ·
-
in-progress
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
tableau/server-client-python#1829 · 1 comentário ·
-
enhancement gap needs investigation
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
tableau/server-client-python#1322 · 1 comentário ·
-
[Type2] Allow Incremental Refresh type schedules to be added via `server.schedules.add_to_schedule` Abertahelp wanted Server-Side Enhancement ui-exists
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
tableau/server-client-python#1101 · 3 comentários ·
-
enhancement good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
tableau/server-client-python#783 · 5 comentários ·
Todas as issues de tableau/server-client-python
Issues semelhantes
-
area: harness bug status: needs-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Human-Agent-Society/reef#625 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 80/100
learningequality/kolibri#15351 · 2 comentários ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Name consistency Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
eellak/triplestore#65 · 1 comentário ·