Skip to content

Conversation

henriquemattos
Copy link
Contributor

@henriquemattos henriquemattos commented Sep 28, 2025

Using native https from NodeJS instead of SPDY, which is not supported by Node >= 24.

According to NodeJS Releases, “v22 is about to enter Maintenance and v24 is about to become Active”. Unfortunately, this is a breaking change for UI5 CLI serve when using --h2, since we use spdy. According to the code comments, we use spdy because of Node v8.4.0 😬.

Tested the upgrade using Firefox Developer Edition (latest) on macOS 26 and NodeJS v24.9.0 with:

ui5 serve --verbose --port 1987 -o index.html --h2 --key ./infrastructure/certs/localhost.key --cert ./infrastructure/certs/localhost.crt --accept-remote-connections

Fixes: #327


Thank you for your contribution! 🙌

To get it merged faster, kindly review the checklist below:

Pull Request Checklist

Copy link

cla-assistant bot commented Sep 28, 2025

CLA assistant check
All committers have signed the CLA.

@flovogt
Copy link
Member

flovogt commented Sep 29, 2025

@henriquemattos Thanks a lot for your contribution.

Note: This PR is a breaking change and would also drop HTTP/2 support but ensure HTTPS is working again.

@henriquemattos
Copy link
Contributor Author

I see your point, @flovogt . Thank you for the feedback. I see HTTP/2 in development environment as a “nice-to-have” feature, while SSL is, at least for Firefox, a “must-have”. The default settings of the browser uses https even for localhost. Besides, changing the protocol on the frontend would require a CORS, Proxy or change to the backend too.

Based on your comment, I don’t understand what are the next steps?!

kind regards,
Henrique

@flovogt flovogt self-assigned this Sep 30, 2025
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.

Use native http2 instead of spdy

2 participants