New Windows PT time zone results in UI error due to unresolved time zone
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 66/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- javascript
- Domain
- frontend
Research direction
Start in the Airflow web UI code where the selected time zone is resolved and used to render times; the report points to an unguarded selectedTimezone.toUpperCase() call but gives no source file or test. Reproduce with the Windows Pacific Time zone and cleared browser local storage, then find or add a test for an unresolved OS time zone. Done means time-rendering pages load with a safe fallback instead of crashing.
Written by the indexing model from the issue text.
Description
Under which category would you file this issue?
Airflow Core
Apache Airflow version
3.1.7
What happened and how to reproduce it?
After recently switching my time zone from PDT to the new unified Pacific Time (PT) time zone, I received the following error after logging in:
TypeError: Cannot read properties of undefined (reading 'toUpperCase')
at gd (index-DMFz1oOG.js:584:8179)
at Dft (index-DMFz1oOG.js:70:34718)
at Jit (index-DMFz1oOG.js:70:63847)
at Dvn (index-DMFz1oOG.js:70:74419)
at a1n (index-DMFz1oOG.js:70:108782)
at fSr (index-DMFz1oOG.js:70:107817)
at Ttt (index-DMFz1oOG.js:70:107643)
at r1n (index-DMFz1oOG.js:70:104696)
at m1n (index-DMFz1oOG.js:70:116348)
at MessagePort.vn (index-DMFz1oOG.js:55:1683)
This appears to be caused by stale time zone information in local storage and an unguarded call to selectedTimezone.toUpperCase() where selectedTimezone is undefined due to being unable to resolve the OS time zone to an IANA name.
Reproduction:
- Set your Windows time zone to Pacific Time (UTC -7:00)
- Clear browser local storage for the Airflow server
- Log in to Airflow
- Any page that renders the Time crashes
I'm able to work around this by changing the time zone for my user in the Airflow UI.
What you think should happen instead?
The UI should fall back to a known value when it fails to resolve the OS time zone, e.g., UTC. Any page that renders the time should not crash.
Operating System
Windows 11 64-bit
Deployment
Official Apache Airflow Helm Chart
Apache Airflow Provider(s)
No response
Versions of Apache Airflow Providers
No response
Official Helm Chart version
Not Applicable
Kubernetes Version
Not Applicable
Helm Chart configuration
No response
Docker Image customizations
No response
Anything else?
No response
Are you willing to submit PR?
- Yes I am willing to submit a PR!
Code of Conduct
- I agree to follow this project's Code of Conduct
- Dominant language
- Python
- Stars
- 47.3k
- Forks
- 18.1k
- Avg merge
- 18h 16m
- Merged PRs (30d)
- 647
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- Ships a Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/airflow
-
update virtualenv to 21.14.12 or later to resolve emSign security issuePossibly taken A pull request linked to this issue is open or already merged. Openkind:bug needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
apache/airflow#75858 · 1 comment ·
Maintainers usually reply within 1 day
-
`@task.pyspark` does not template `config_kwargs` and `conn_id`Possibly taken @sdaberdaku claimed this 2 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 79/100
Maintainers usually reply within 1 day
-
Trigger UI clamps Param min/max instead of rejecting out-of-range valuesPossibly taken @muditagrawal2007 claimed this 2 days ago. Openkind:bug needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
apache/airflow#75747 · 1 comment ·
Maintainers usually reply within 1 day
-
Deadline Alert: No support for SmtpNotifierPossibly taken @LiaoAB claimed this 4 days ago. Openkind:bug needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
apache/airflow#75452 · 1 comment ·
Maintainers usually reply within 1 day
-
RedactedIO leaks secret bytes to stdoutPossibly taken @ShivanshhhG claimed this 10 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
apache/airflow#75069 · 1 comment ·
Maintainers usually reply within 1 day
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Graphify-Labs/graphify#4860 · 1 comment ·
Maintainers usually reply within 1 day
-
:bug: Bug
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
scikit-image/scikit-image#8377 ·
Maintainers usually reply within 1 day
-
Ingredient import validation accepts negative nutrient values and NaNPossibly taken @LukeSekai claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
wger-project/wger#2564 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
apache/ponymail-foal#351 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 68/100
Maintainers usually reply within 4 days