Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert to disregarding the mount path for /livereload/ endpoint #2804

Merged
merged 1 commit into from Mar 26, 2022

Conversation

oprypin
Copy link
Contributor

@oprypin oprypin commented Mar 26, 2022

Previously there was a pull request "Move livereload endpoint under the mount path (#2740)"
and it definitely makes sense as a way to make livereload work behind a proxy.

However, this change also caused a major annoyance when working on several different sites.
Steps to reproduce:

  1. serve one site (needs to have site_url customized)
  2. open the site in a browser
  3. stop serving it but keep the browser
  4. serve another site with a different mount point
  5. observe spam of "404" errors in the new terminal (also in the old browser but that's whatever)

So, this reverts the observable effect of commit fafdcc2 by @robertaistleitner

Previously there was a pull request "Move livereload endpoint under the mount path (mkdocs#2740)"
and it definitely makes sense as a way to make livereload work behind a proxy.

However, this change also caused a major annoyance when working on several different sites.
Steps to reproduce:

1. `serve` one site (needs to have `site_url` customized)
2. open the site in a browser
3. stop serving it but keep the browser
4. `serve` another site with a different mount point
5. observe spam of "404" errors in the new terminal (also in the old browser but that's whatever)

So, this reverts the observable effect of commit fafdcc2
@oprypin oprypin merged commit b727215 into mkdocs:master Mar 26, 2022
@oprypin oprypin deleted the unlive branch March 26, 2022 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant