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

[dev] hot-reloading broken #1737

Closed
jameshadfield opened this issue Jan 29, 2024 · 2 comments · Fixed by #1738
Closed

[dev] hot-reloading broken #1737

jameshadfield opened this issue Jan 29, 2024 · 2 comments · Fixed by #1738
Labels
bug Something isn't working

Comments

@jameshadfield
Copy link
Member

Running auspice develop ... works fine initially, but as soon as the code rebuilds it is no longer viewable in the browser. Firefox displays "The page isn’t redirecting properly. Firefox has detected that the server is redirecting the request for this address in a way that will never complete". It looks like we're getting into a infinite loop of 302 redirects to the same page ("/").

How to reproduce

Steps to reproduce the current behavior:

  1. auspice develop
  2. load localhost:4000 - note that all works as expected
  3. Change some code, you should see "webpack building..." in the terminal
  4. Refresh localhost:4000

Possible solution

The bug is present at current master (40befc6).
Bisecting indicates the bug was introduced in #1733

@jameshadfield jameshadfield added the bug Something isn't working label Jan 29, 2024
@victorlin
Copy link
Member

I haven't looked at this in depth yet but we can just revert 798f3ee for the time being. It's not an important change

@victorlin
Copy link
Member

victorlin commented Jan 30, 2024

Seems to be weird interaction between Hot Module Replacement and output.clean. A search led me to webpack/webpack-dev-middleware#861. Using CleanWebpackPlugin is a known workaround. I'll make a PR to revert.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants