Support publicPath in vue.config.js

[ulich]

Fixes #97

[dennisreimann]

Some minor additions:

Implementing prepareUrlForRouting like this and moving it below the import statement in entry-server.js would better fit the coding style of this project and also ensure that the base url works without the trailing slash:

const prepareUrlForRouting = url => {
  const { BASE_URL } = process.env
  return url.startsWith(BASE_URL.replace(/\/$/, ''))
    ? url.substr(BASE_URL.length)
    : url
}

We could also change this line in dev-server.js to point to the correct entry url of the app:

const url = `http://localhost:${config.port}${service.projectOptions.publicPath}`

When testing this, I found the changes in 2e729de to not be necessary to make the publicPath option work with this plugin. I might be missing something, as the app I'm trying it out with is not very big yet. In case this is necessary I think we should also prepend the publicPath in this line like that:

app.get(`${publicPath}*`, (req, res, next) => {
  if (config.skipRequests(req)) {
    return next()
  }
  ssr(req, res)
})

[ulich]

I will adapt my PR with your remarks.

About:

When testing this, I found the changes in 2e729de to not be necessary to make the publicPath option work with this plugin.

This only happens when running the production build. The requests to download the javascript bundle all return the html content of the page instead of the static files.

[dennisreimann]

I have this PR running in our project and tested it to the extend of our needs. AFAIK this seems good to go 👍

[dennisreimann]

fyi: While we are waiting for this to get merged I've published a fork on npm, which includes this and other PRs.

Looking forward to seeing this merged!

[Akryum]

👍