vercel/serve

Allow removal of a header

Open

#438 geöffnet am 21. Juni 2018

Auf GitHub ansehen
 (8 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)TypeScript (8.811 Stars) (719 Forks)batch import
help wantedpatch

Beschreibung

I'd like to be able to set a default header and then override it to nonexistance. When I do the below, a header with null shows up, versus the header disappearing. Ditto with max-age=0. And when I do "headers": [], it gets 86400 from the first rule.

image

image

cc @leo

Contributor Guide

Tech Stack
typescriptnodejs
Domain
backendcli
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Basic HTTP headers knowledgeNode.js environmentServe project setup
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
Examine the serve source code, particularly the header handling middleware and configuration schema. Look for how default headers are applied and if there is existing logic for overriding. Check the comments on this issue for any proposed solutions. Consider modifying the configuration parsing to allow an empty value or special key to remove a header.