Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

38C3: set refresh_interval to 30 minutes #398

Merged
merged 1 commit into from
Dec 14, 2024

Conversation

schildbach
Copy link
Contributor

38C3 Fahrplan changes happen frequently and often last-minute. The 30 minute value is copied from last year's 37C3, which apparently went without issues.

The server currently doesn't support If-None-Match properly, which is why CI will fail.

I'm proposing this as a PR anyway, so we can merge as soon as either

  1. the issue is fixed server-side or
  2. the admins approve this PR (in which case the CI should be ignored imho)

Copy link

Changed: 38C3
Error: Schedule 38C3 refresh_interval set below 1d while server never sends HTTP 304s
Error: File validation failed! :-( Please correct the issues listed above.

@psy
Copy link

psy commented Dec 13, 2024

Should be fixed:

$ curl -iv -H "If-None-Match: 62a312ab70d432ba4e985985c42673f951e37a2b" https://fahrplan.events.ccc.de/congress/2024/fahrplan/schedule/export/schedule.xml

[..]
HTTP/2 304
[..]

@Wilm0r
Copy link
Owner

Wilm0r commented Dec 14, 2024

Terrific, thank you!

@Wilm0r Wilm0r merged commit 9c9c50d into Wilm0r:master Dec 14, 2024
1 check failed
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.

3 participants