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

Nodejs download page is acting weird #5102

Closed
zhaoqiu-china opened this issue Mar 11, 2023 · 12 comments
Closed

Nodejs download page is acting weird #5102

zhaoqiu-china opened this issue Mar 11, 2023 · 12 comments
Labels

Comments

@zhaoqiu-china
Copy link

Version

No response

Platform

No response

Subsystem

No response

What steps will reproduce the bug?

  1. go to https://nodejs.org/en/download/
  2. click "Current"
  3. click Windows Installer. (My pc os is win10-x64 and "Windows Installer" has a href linkd "https://nodejs.org/dist/v19.7.0/node-v19.7.0-x64.msi")
  4. I expect that x64.msi will be downloaded, however it's x86.msi

How often does it reproduce? Is there a required condition?

No response

What is the expected behavior?

No response

What do you see instead?

image

Additional information

No response

@richardlau richardlau transferred this issue from nodejs/node Mar 11, 2023
@SEWeiTung SEWeiTung added the bug label Mar 12, 2023
@SEWeiTung
Copy link
Contributor

SEWeiTung commented Mar 12, 2023

Not only for Current,But LTS as well has the issue in the download page
Link to x64 but download for x86....

@SEWeiTung
Copy link
Contributor

SEWeiTung commented Mar 12, 2023

1678592034670

The reason may be the "x86" is fixed in the download link....And it is dynamically changed by js behind. However due to some reason it changes but in fact it isn't....I'll investigate why and fix the bug

@ovflowd
Copy link
Member

ovflowd commented Mar 12, 2023

@MaledongGit, your PR does not fix the issue. The issue is actually about the OS detection we have. We cannot reliably detect if the Windows OS is x64 or x86, and that's what needs to be fixed.

@ovflowd
Copy link
Member

ovflowd commented Mar 12, 2023

The actual PR fixing this issue is #5025

@ovflowd
Copy link
Member

ovflowd commented Mar 12, 2023

The reason may be the "x86" is fixed in the download link....And it is dynamically changed by js behind. However due to some reason it changes but in fact it isn't....I'll investigate why and fix the bug

It is changing it, just changing it incorrectly. We assess the platform architecture by looking up the user agent and some meta info. In some browsers, we don't assess that reliably.

@zhaoqiu-china, on your screenshot, the Error attached in the console has nothing to do with the download issue. It is just the behaviour of next/link trying to prefetch that URL; I will change that component not to use next/link :)

@ovflowd
Copy link
Member

ovflowd commented Mar 12, 2023

For reference: This is fixed by #5025 and #5106

@SpeedyCodeYT

This comment was marked as off-topic.

@ovflowd
Copy link
Member

ovflowd commented Mar 12, 2023

@SpeedyCodeYT, what are you even talking about? 😅

@SEWeiTung
Copy link
Contributor

The reason may be the "x86" is fixed in the download link....And it is dynamically changed by js behind. However due to some reason it changes but in fact it isn't....I'll investigate why and fix the bug

It is changing it, just changing it incorrectly. We assess the platform architecture by looking up the user agent and some meta info. In some browsers, we don't assess that reliably.

@zhaoqiu-china, on your screenshot, the Error attached in the console has nothing to do with the download issue. It is just the behaviour of next/link trying to prefetch that URL; I will change that component not to use next/link :)

Got it, I found it later.
Thanks for your guidance and I've removed my PR here.

@ovflowd
Copy link
Member

ovflowd commented Mar 13, 2023

@MaledongGit no worries! I appreciate your help and intentions. I hope my explanations on your PR made sense ;)

@SEWeiTung
Copy link
Contributor

Hi @zhaoqiu-china

This bug should be fixed now, could you please check and close the issue?

@zhaoqiu-china
Copy link
Author

Hi @zhaoqiu-china

This bug should be fixed now, could you please check and close the issue?

Ok, thanks for your efforts!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants