UserItem.CSVImport.create_from_file: wrong file-extension check

Offen Anfängerfreundlich

#1,865 erstellt am 17.8.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 70/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Ruhig
Tech-Stack: python
Bereich: api

Rechercherichtung

Beginne in users_endpoint.py bei Users.create_from_file und überprüfe dessen Deprecation-Verhalten sowie die aktuellen Aufrufer. Überprüfe die aufgeführten Pfade für Kleinbuchstaben, Großbuchstaben, Teilstrings und Nicht-CSV und bestätige anschließend, dass der gewählte Ansatz – die Validierung während des Deprecation-Zeitraums zu korrigieren oder dies bis zur Entfernung aufzuschieben – der Projektentscheidung entspricht.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Users.create_from_file (in users_endpoint.py) validates the file with "csv" not in filepath before parsing. That test:

accepts non-CSV files whose path merely contains the substring csv (e.g. report.csv.bak, mycsv.txt, /home/csvuser/data.json)
rejects valid files whose extension is uppercase (e.g. USERS.CSV)

Neither matches the error message ("Only csv files are accepted").

Fix would be Path(filepath).suffix.lower() == ".csv".

Note: create_from_file is already deprecated (emits DeprecationWarning and is being replaced by bulk_add). Two paths:

Patch under the deprecation window so the last few callers get correct behavior until removal.
Just delete the method when the deprecation window closes and don't touch it in the meantime.

Filed as follow-up to a review comment on PR #1812 (out of scope for that PR's refactor).

Vorherrschende Sprache: Python
Sterne: 716
Forks: 446
Ø Merge: 8 T. 8 Std.
Gemergte PRs (30 T.): 2

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus tableau/server-client-python

security: _validate_import_line_or_throw logs credential fields at DEBUG level Offen

in-progress

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

tableau/server-client-python#1829 · 1 Kommentar · Erstellt am 23.7.2026
Add usageStatistics Param to view.get_by_id Method Offen

enhancement gap needs investigation

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

tableau/server-client-python#1322 · 1 Kommentar · Erstellt am 14.11.2023
[Type2] Allow Incremental Refresh type schedules to be added via `server.schedules.add_to_schedule` Offen

help wanted Server-Side Enhancement ui-exists

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

tableau/server-client-python#1101 · 3 Kommentare · Erstellt am 06.9.2022
Delete default permissions does not take in a list Offen

enhancement good first issue

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

tableau/server-client-python#783 · 5 Kommentare · Erstellt am 22.1.2021
[security feature] Require explicit opt-in for allowing redirect http->https Offen

enhancement needs investigation

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100

tableau/server-client-python#1879 · Erstellt am 10.9.2026

Alle Issues in tableau/server-client-python