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.

Sign up for GitHub

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

Jump to bottom

fix(dev): apply headers from route rules for static assets #2814

Open
wants to merge 2 commits into
base: v2
Choose a base branch
from
Open

fix(dev): apply headers from route rules for static assets #2814

wants to merge 2 commits into from

Commits on Oct 25, 2024

  1. Apply headers from routeRules in the dev server. 

    - These otherwise fail to be applied to public
  assets (see nitrojs#2749).
- This change replies them redundantly in the
  other cases (where they were already being
  applied), but the alternative would require
  duplicating or reversing the URL / file path
  logic from the "send" package.

fixes nitrojs#2749
    @peterthenelson
    peterthenelson committed Oct 25, 2024
    Configuration menu
    Copy the full SHA
    572d91f View commit details
    Browse the repository at this point in the history

  2. Fix lint errors

    @peterthenelson
    peterthenelson committed Oct 25, 2024
    Configuration menu
    Copy the full SHA
    704877a View commit details
    Browse the repository at this point in the history