Naive datetime values are interpreted in the host's local time zone

Aperta

#50 aperta il 14/9/2026 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 5/5
Tempo stimato: Più di una settimana
Idoneità per principianti: 35/100

Tipo di issue: Bug
Chiarezza: Abbastanza chiara
Stato di attività: Attiva
Stack tecnologico: python, rust
Ambito: api, backend

Direzione di ricerca

Inizia da RustyPyType::try_into_value e analizza la conversion di datetime, quindi esamina il riferimento dell'API Python, la guida introduttiva rapida, i tutorial e gli esempi nella docstring di Context.add_variable. Confronta le tre policy proposte per i valori naive di data e ora con il comportamento esistente e la direzione del progetto. Il lavoro è completato quando la policy è stata concordata, implementata se necessario e tutti gli esempi della documentazione interessati la riflettono in modo coerente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Behaviour

Converting a Python value to a CEL timestamp (in RustyPyType::try_into_value) handles a timezone-aware datetime correctly, but a naive one is passed through chrono::Local, so the same datetime(2026, 1, 1, 12) is a different instant on a UTC server and on a developer's laptop. CEL timestamps are absolute instants, and every timestamp(...) literal an expression builds is UTC, so comparisons like created > timestamp("2026-01-01T00:00:00Z") silently depend on TZ.

datetime.now() (naive) is the common way to hit this. The quick-start guide, the tutorials and the Context.add_variable docstring all use naive datetime.now() as the example value, so the docs currently teach the footgun.

Options

Keep local, document loudly. Matches Python's own convention (naive.timestamp() assumes local time). Least disruptive; add a warning box to the Python API reference and the datetime section of the tutorial, and switch the doc examples to datetime.now(timezone.utc).
Treat naive as UTC. Deterministic across machines and matches what most server code means. Silent behaviour change for anyone relying on option 1.
Reject naive datetimes with a ValueError that says to attach a tzinfo. Loudest and safest; a breaking change for existing callers.

My lean is 1 now (with the doc examples fixed) and 3 at the 1.0 boundary, since a policy engine that gives different answers per host TZ is the kind of bug that only shows up in production. Opinions welcome before anything changes.

Lingua principale: Python
Stelle: 43
Fork: 4
Merge medio: 9h 57m
PR unite (30g): 14

Guida per i contributori

Apri la guida per i contributori

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.