11ty / 11ty/dev-server

Remove finalhandler dependency

Open
#133 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
109
Forks
20
Avg merge
1m
Merged PRs (30d)
1

Description

This dependency contributes a bit of bulk to this package: https://npmgraph.js.org/?q=@11ty/eleventy@4.0.0-alpha.7#select=finalhandler%402.1.1

Image

And appears to be unnecessary, and kind of an inconvenience even - it has one bug with a workaround and an unwanted CSP policy. I think it could be removed and replaced with the barebones logic that we want instead.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating where finalhandler is imported or used and where the dependency is declared. Review the existing workaround and unwanted CSP behavior, then verify that finalhandler is removed while the server retains the intended barebones response handling.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
devtools, web-dev
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.