Parcel report dev server overwriting headers set on proxyrc.js middlewares

[vitorcodesalittle]

🐛 bug report

🎛 Configuration (.babelrc, package.json, cli command)

.proxyrc.js

module.exports = function(app) {
  app.use((req, res, next) => {
    res.setHeader('Cross-Origin-Embedder-Policy', 'unsafe-none')
    next()
  })
}

.babelrc.json:

{
  "presets": [
    "@parcel/babel-preset-env",
    "@babel/preset-react"
  ],
  "plugins": [
    "babel-plugin-transform-glob-import",
    "@parcel/babel-plugin-transform-runtime",
    ["import", { "libraryName": "antd", "libraryDirectory": "lib", "style": false}, "antd"],
    ["import", { "libraryName": "antd-new", "libraryDirectory": "lib", "style": "css"}, "antd-new"]
  ]
}

cli command:
parcel public/index.html --port 4000 --log-level verbose

🤔 Expected Behavior

Responses should have the header as defined on the middleware.

😯 Current Behavior

Response always with Cross-Origin-Embedder-Policy: require-corp. Probably also happens with other headers being set by

parcel/packages/reporters/dev-server/src/Server.js

Line 33 in e63235f

function setHeaders(res: Response) {

💁 Possible Solution

I believe is just a matter of the order of the middlewares being called. I would be glad try a PR once the issue is confirmed to be an issue.

🔦 Context

I wish I could have a more flexible development server.

🌍 Your Environment

Software	Version(s)
Parcel	2.0.0-nightly.720
Node	14.17.0
npm/Yarn	6.14.13
Operating System	Fedora 34

[mischnic]

I think it would make sense to call setHeaders before the proxyrc is applied 👍

[HenrikJoreteg]

This bit me.

I was just including Stripe and was surprised it didn't work when updating from Parcel 1 to Parcel 2. It's kind of uninituitive, IMO that something as benign as lazily loading 3rd party JS doesn't work.

[mischnic]

This has been reverted: #6789

[HenrikJoreteg]

@mischnic thanks 😊