Skip to content

feat!: vite dev default port is now 5173#8148

Merged
patak-cat merged 1 commit intomainfrom
feat/vite-port-riddle
May 12, 2022
Merged

feat!: vite dev default port is now 5173#8148
patak-cat merged 1 commit intomainfrom
feat/vite-port-riddle

Conversation

@patak-cat
Copy link
Member

Description

Switch from 3000 to 5173 for the vite dev server port, avoiding a common port as Vite's default.

We already migrated from 5000 to 4173 for the preview server in a prev minor so they will be aligned now.


What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

@patak-cat patak-cat changed the title feat: vite dev default port is now 5173 feat!: vite dev default port is now 5173 May 12, 2022
Copy link
Member

@Shinigami92 Shinigami92 left a comment

Choose a reason for hiding this comment

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

🚀

@patak-cat patak-cat merged commit 1cc2e2d into main May 12, 2022
@patak-cat patak-cat deleted the feat/vite-port-riddle branch May 12, 2022 14:47
@antfu antfu added this to the 3.0 milestone May 13, 2022
sapphi-red added a commit to sapphi-red/vite that referenced this pull request May 31, 2022
@MarvinJWendt
Copy link

Was this intentionally choosen, because it looks kinda like VITE? 😄

5173
VITE

@zhengxiaoyao0716
Copy link

Port 5173 may be used by Hyper-V due to Windows mistakenly setting the reserved port range to 1025-15000.
Of course it's not the responsibility of vite, just a reminder to prevent others meting the problem.

Reference:
https://stackoverflow.com/questions/48478869/cannot-bind-to-some-ports-due-to-permission-denied/62359555#62359555
https://learn.microsoft.com/en-us/troubleshoot/windows-server/networking/default-dynamic-port-range-tcpip-chang#introduction
docker/for-win#3171 (comment)

@dbry23
Copy link

dbry23 commented Feb 4, 2025

I thought it looked more like SITE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants