[GH-ISSUE #351] Ignore new HTTP Priority header #273

Closed
opened 2026-05-05 03:57:42 -06:00 by gitea-mirror · 2 comments
Owner

Originally created by @binwiederhier on GitHub (Jul 1, 2022).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/351

Cloudflare apparently adds a Priority: u=... header to requests, which ntfy rejects with 500 invalid priority (should be a 400). This is a new HTTP standard (https://datatracker.ietf.org/doc/html/draft-ietf-httpbis-priority) that is emerging.

We should ignore header values that start with u=. It is quite unfortunate that the name conflicts like this.

Originally created by @binwiederhier on GitHub (Jul 1, 2022). Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/351 Cloudflare apparently adds a `Priority: u=...` header to requests, which ntfy rejects with `500 invalid priority` (should be a 400). This is a new HTTP standard (https://datatracker.ietf.org/doc/html/draft-ietf-httpbis-priority) that is emerging. We should ignore header values that start with `u=`. It is quite unfortunate that the name conflicts like this.
gitea-mirror 2026-05-05 03:57:42 -06:00
Author
Owner

@binwiederhier commented on GitHub (Jul 1, 2022):

Fixed in 95bd876be2

<!-- gh-comment-id:1172351556 --> @binwiederhier commented on GitHub (Jul 1, 2022): Fixed in https://github.com/binwiederhier/ntfy/commit/95bd876be21fbff56fc0082725a69d6cb5a18a7c
Author
Owner

@binwiederhier commented on GitHub (Jul 1, 2022):

This ticket (https://github.com/binwiederhier/ntfy/issues/353) has a full dump of the Cloudflare request showing the Priority: u=3 header.

<!-- gh-comment-id:1172537581 --> @binwiederhier commented on GitHub (Jul 1, 2022): This ticket (https://github.com/binwiederhier/ntfy/issues/353) has a full dump of the Cloudflare request showing the `Priority: u=3` header.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/ntfy#273
No description provided.