Use web page address when making request to reload proxy (#625)

* Use web page address when making request to reload proxy Signed-off-by: Alexandra Fishova <yopoyka@yopoyka.com> * Create event source with just a path because browser can handle address on its own. Signed-off-by: Alexandra Fishova <yopoyka@yopoyka.com> --------- Signed-off-by: Alexandra Fishova <yopoyka@yopoyka.com>
air-verse · Sep 21, 2024 · 6761441 · 6761441
1 parent f4f7c3d
commit 6761441
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/runner/proxy.go b/runner/proxy.go
@@ -66,10 +66,7 @@ func (p *Proxy) injectLiveReload(resp *http.Response) (string, error) {
 		return page, nil
 	}
 
-	script := fmt.Sprintf(
-		`<script>new EventSource("http://localhost:%d/internal/reload").onmessage = () => { location.reload() }</script>`,
-		p.config.ProxyPort,
-	)
+	script := `<script>new EventSource("/internal/reload").onmessage = () => { location.reload() }</script>`
 	return page[:body] + script + page[body:], nil
 }
 

diff --git a/runner/proxy_test.go b/runner/proxy_test.go
@@ -190,7 +190,7 @@ func TestProxy_injectLiveReload(t *testing.T) {
 				},
 				Body: io.NopCloser(strings.NewReader(`<body><h1>test</h1></body>`)),
 			},
-			expect: `<body><h1>test</h1><script>new EventSource("http://localhost:1111/internal/reload").onmessage = () => { location.reload() }</script></body>`,
+			expect: `<body><h1>test</h1><script>new EventSource("/internal/reload").onmessage = () => { location.reload() }</script></body>`,
 		},
 	}
 	for _, tt := range tests {