Skip to content

Fix internal host notification#17

Open
bao7uo wants to merge 2 commits intomdsecactivebreach:masterfrom
bao7uo:patch-1
Open

Fix internal host notification#17
bao7uo wants to merge 2 commits intomdsecactivebreach:masterfrom
bao7uo:patch-1

Conversation

@bao7uo
Copy link
Copy Markdown

@bao7uo bao7uo commented Nov 6, 2020

Changed notification to config.Server.Host to match what is happening on line 55

Changed notification to config.Server.Host to match what is happening on line 55
Copy link
Copy Markdown

@yezz123 yezz123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Configure the host from host to local one?

@bao7uo
Copy link
Copy Markdown
Author

bao7uo commented Nov 24, 2020

See the internal server starts on config.Server.Host - so the listen notification message should not be hardcoded to 127.0.0.1

	server := &http.Server{
		Addr:    fmt.Sprintf("%s:%d", config.Server.Host, config.Server.InternalPort),
		Handler: route,
	}
	server.ListenAndServe()

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.

3 participants