New proxy library in 1.15 isn't backwards-compatible

[benweissmann]

The new proxy library used in 1.15 uses a different glob syntax, so this configuration:

{
  devServer: {
    proxy: {
      "*": {
        target: proxyUrl,
        secure: false,
      },
    },
  },
};

works in 1.14, but not 1.15. In 1.15, it needs to be ** instead of *.

Should we add a warning or something to aid in upgrading?

[leobalter]

I've just found this same issue, @mzgoddard helped me. This was apparently introduced migrating from http-proxy to http-proxy-middleware. (7b3f12e)

It looks like this '*' pattern is pretty common, so this might be an annoying breaking change in the semver releases.

In the meanwhile can we set a translation for this pattern to match ' **'? This would be easy to remove releasing the version 2 already in beta.

[leobalter]

It looks like this is a duplicate of #561 and #560

[leobalter]

at least '**' is a compatible glob in 1.14.1, so it won't break someone's instance by any missed version bump on the webpack-dev-server version.

[Yusoo]

Also in version 2.1.0-beta.0

[joshhunt]

:/

This is a fairly annoying breaking change to just automatically get - I would appreciate at least a warning if using *. Preferably, automatically mapping * to ** would be a lot more helpful.

[tahone]

Changing * or /* to ** solved the HPM the error but bypass doesn't do anything anymore. Everything is sent to the proxy.

[s0meone]

I've used the custom matching option of the proxy middleware instead of the bypass option.

...
    proxy: [
      {
        context: function(pathname, req) {
          // only proxy the api and the login post call to the node backend
          return pathname.match("^/api") || (pathname.match("^/login") && req.method === "POST")
        },
        target: {
          port: 3000
        }
      }
    ]
 ...

Hope this helps anyone.

[SpaceK33z]

Hm, sorry that this happened. In PR #359, which implemented the switch from http-proxy to http-proxy-middleware, there seemed to be only one breaking change. This was fixed for backwards compatibility with commit 33ebd1f.

As for the bypass option not working, let's continue that discussion in #560.

[SpaceK33z]

As of 314048e, using * as a context should work again. Internally, it will be remapped to **. Can you guys check if that works for you?

The easiest way to test this, is to replace the contents of node_modules/webpack-dev-server/lib/Server.js with this file.

[aknuds1]

I was surprised by this too, only by stumbling across this issue did I understand what had happened. Also, I think the webpack-dev-server documentation is now inconsistent, since the proxy comment in the API section says the following, whereas it should be stipulating ** instead:

// Set this if you want webpack-dev-server to delegate a single path to an arbitrary server.
// Use "*" to proxy all paths to the specified server.

[SpaceK33z]

@aknuds1, see my comment above. It is getting fixed ;).

[aknuds1]

@SpaceK33z Yeah I saw that, but the webpack-dev-server documentation is still inconsistent, isn't it? It confused me quite a bit when I was re-reading it, to figure out what might be happening.

[SpaceK33z]

Ah yeah, true, that should be fixed now.

[aknuds1]

@SpaceK33z Thanks!

[leobalter]

It works for me! Thanks, @SpaceK33z!

[davidreher]

@Mischi and me are also having issues when using a proxy and HTTPS with own certificates (key, crt and ca).

With 1.14.2 we could do a redirect from localhost/abc to localhost:

module.exports.devServer.proxy = {
  '/abc/**': {
    target: 'https://localhost:8080',
    rewrite: function rewrite(req) {
      req.url = req.url.replace(/^\/abc/, '');
    },
  },

now we are getting

[HPM] PROXY ERROR: UNABLE_TO_VERIFY_LEAF_SIGNATURE. localhost -> https://localhost:8080/abc/

on the console and

Error occured while trying to proxy to: localhost:8080/abc/

inside the browser.

[SpaceK33z]

@davidreher, this could be related to #566. As a workaround, you can use pathRewrite from http-proxy-middleware. If that still doesn't work, try adding secure: false.

[davidreher]

@SpaceK33z pathRewrite was part of the problem. Finally adding secure: false did the trick. Thx for the quick support 👍

[SpaceK33z]

This problem has been fixed with webpack-dev-server@1.15.1. * is now rewritten to ** internally.