Skip to content

Fix Tonight 500: set iers.conf.auto_max_age = None - #187

Open
Thundras wants to merge 1 commit into
bscholer:mainfrom
Thundras:fix/iers-tonight
Open

Thundras wants to merge 1 commit into
bscholer:mainfrom
Thundras:fix/iers-tonight

Conversation

@Thundras

@Thundras Thundras commented Aug 28, 2026

Copy link
Copy Markdown

Summary

Set iers.conf.auto_max_age = None in server/sky.py to fix the Tonight page throwing 500 inside air-gapped / Docker containers. Closes no existing issue but is a self-contained one-liner.

auto_download and iers_auto_url are already disabled; auto_max_age (default 30 days) still caused astropy to raise ValueError: IERS table is more than 30.0 days old because the bundled table can never be refreshed offline.

Test plan

  • pytest tests/ passes locally — 674/682 tests pass; 8 pre-existing failures unrelated to this PR: test_orphan_master_removed (timing bug confirmed on upstream fcb1243), 2x test_scanner_orphans (Windows path-separator bug in _remove_orphans, only visible without -x), 5x test_siril_runtime (no Siril binary on Windows dev machine)
  • npm run check clean from ui/ — 0 errors; 3 pre-existing CSS warnings in playground/themes (unrelated)
  • Manually exercised the affected flow in the browser — Tonight page loads without 500 in a container run with no outbound network
  • N/A — explain why

Notes for the reviewer

The three iers.conf lines are already grouped together; this is a one-line addition to that block. No other files touched.

Offline containers have no network access to refresh the IERS table, so
astropy raises ValueError when the cached table is older than the default
30-day max_age. Setting auto_max_age = None disables the age check and
lets the bundled table be used indefinitely.

Fixes the 500 error on the Tonight page in any air-gapped / Docker setup.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant