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

web-server: Print localhost instead of 0.0.0.0 #8952

Merged
merged 1 commit into from
Jul 22, 2023

Conversation

Tobbe
Copy link
Member

@Tobbe Tobbe commented Jul 22, 2023

When I run yarn rw serve web I see this printed to the console
image

It's not wrong. But when clicking that link I get "page is not secure" warning icon
image

And I don't get the dev tools I get when using localhost instead
(which look like this)
image

The devtools thing might be more of a browser thing, but I also think people are just more used to seeing/using localhost.
So I suggest printing http://localhost:8910 instead.

This issue was introduced with the new web-server package because I switched to just printing the address we get from Fastify.

@Tobbe Tobbe added the release:fix This PR is a fix label Jul 22, 2023
@Tobbe Tobbe added this to the next-release milestone Jul 22, 2023
@Tobbe Tobbe enabled auto-merge (squash) July 22, 2023 19:29
@Tobbe Tobbe merged commit fc07945 into redwoodjs:main Jul 22, 2023
29 of 33 checks passed
@jtoar jtoar modified the milestones: next-release, v6.1.0 Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix This PR is a fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants