Doesn't consider basePath? about connected-next-router HOT 3 OPEN

I can't reproduce it in the examples given, but it's definitely causing weird artifacts in my app. Like redundantly duplicating basePath "/frontend/frontend/...", and subsequently redirecting to it which obviously results in 404. I can't yet figure out why or where this is happening, or whether its my own app or this library...

But I've found a workaround that seems to avoid the issue of unnecessary redirection and path changes.

By patching next/router:

import Router from 'next/router'
import { createRouterMiddleware, ConnectedRouter } from 'connected-next-router';

/* Make `Router.asPath` return with `basePath` prefixed (if absent) */
const patchedRouter = new Proxy(Router, {
  get: (Router, key, receiver) => {
    if (key === 'asPath') {
      const { basePath, asPath } = Router
      const replaced = asPath.includes(basePath) ? asPath : basePath + asPath
      return replaced
    } else {
      return Reflect.get(Router, key, receiver)
    }
  }
})

createRouterMiddleware({ Router: patchedRouter })

<ConnectedRouter Router={patchedRouter}/>

from connected-next-router.

Hi @laggingreflex, I'm currently sick and can't investigate the issue at the moment. I should be able to next week.

from connected-next-router.

I'm also seeing this behavior, but only when using query parameters. Otherwise great work!

from connected-next-router.