-
-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
fix: update preview port #6330
fix: update preview port #6330
Conversation
…to chore/update-port
At a team meeting we thought about 5173 |
They thought 😅 |
Seems that @bluwy like it too ^^ |
I like typing the url in the browser and its faster to write a thousand number with zeroes |
Four equally-spaced odd numbers, that can be joyfully typed as hopscotch, it'll be a huge missed opportunity! In all seriousness, I'm indifferent with either port numbers too. But it looks like a fun easter egg. |
yes but not if you use a different browser than usual, use incognito or have cleared history |
V -> 5 sounds like quite a leap. 4 at least is as upside down V in some sense, so I'd go with 4173 before 5173. Both sound a little hard to remember though. I suppose it's printed to the console, but I would still lean towards 4000
Neither is 5000 and it still sounds like they hijacked it. I hope they didn't grab any others that's we're unaware of 😄 |
I prefer 4173 over 5173 (It reads better)As a normal user, I think Shinigami92 is onto something. I really love the idea of the port number being used to express branding. I also agree that most users probably just click the logged link in the console. (I know I do) Thinking long term, having a dedicated port number (although unofficial) would just make sense for the common user. Who knows that port numbers, unlike domains, are really more of a convention, than a rule. Check out Port 4000 on Wikipedia's list of port numbers. All of this is just community conventions across the entire internet. Who cares about what corporate says, you are and should think Open Source. Honestly guys, Vite is awesome (and has crazy potential). |
We are talking about the preview server here. It looks inconsistent to me that the dev server is going to remain 3000 and the preview will now be a random looking number |
Maybe a twitter poll? 😂 |
I didn't know that it is only the preview port |
there is some history around using 8080 for custom http services. so what about 8050 or 5080 ? keeps the old 50 from 5000 and adds 80 to show it's http. |
Can we close this? (Like Shini, I also agree that the preview port doesn't matter and can be any number.) |
+1 on closing, but for the reason mentioned by @patak-dev, which is that dev and preview ports should be consistent |
But this PR is changing the preview port to 4000, which is consistent with the dev server at 3000. And @junaga, if we could stay at 5000, that would be great but now users in MacOS are having issues when using that port number so changing it seems correct here. |
Ah, I did not realize preview and dev use different ports today. We use 3000 for both for SvelteKit, which I think is easier to remember since you don't have to remember two different ports. Is there any reason not to use the same port for both? |
From what I understand, having two ports could enable you to check both the dev server and the preview server at the same time without extra configuration (in case you want to compare something between the two). It is also interesting to have different caches in the browser for each, no? |
Yeah. I've never felt the need to compare and you can override the port if you do have that need. The cache thing might be more of an issue though if you use service workers. We've had service worker caching be an issue even between different apps because all the browser sees is |
We talked in the last team meeting about the port number. Looks like we are going to use 5173 for the dev server, and change 3000 to it when we release Vite 3.0 in a few months (at the end of life of Node v12). So we should change the preview server from 5000 to a number that is coherent with 5173. @Niputi, from the list you linked, it looks like 4173 is free. Would you help to confirm that this would be a good port number and update the PR if it is? |
port number updated |
Just one small comment about mobile with There is this argument of not being able to quickly open the port (it has to be typed in manually). Maybe something similar like Google added the "Google Lens" icon into the Android Homescreen search bar. I open all my QR codes with it. |
@junaga do you suggest adding vite-plugin-qrcode to core? If that plugin is working well for you, I think it is fine to avoid having to maintain it here |
Oh no no! That plugin is terrible, (it works). I just wanted to mention the option. Logging a QR code when |
Description
fix: #5707
port 4000 is not listed on https://support.apple.com/en-us/HT202944
have chosen 4000 as it looks nice being a 1000 more than dev
Additional context
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).