punycode depreciation

[britkat1980]

v3 of GivTCP uses Alpine3.19 and is now giving a depreciation warning for punycode module:

(node:63) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)

[DanielGallo]

@britkat1980 - I've made this pull request: britkat1980/giv_tcp#259. This is to replace the serve npm module with http-server (to resolve the punycode error).

I also fixed the WebDashboard submodule - it was pointing to an outdated/nonexistent commit, causing it to render a really old version of the web dashboard.

I did a test build of GivTCP image locally, and the web dashboard renders with http-server but I couldn't fully test this - ports 6345, 6346, etc. for the inverter data aren't exposed by default. I also had to add port 3000 as being exposed so I could reach the web dashboard. Perhaps you can fully test it on your end?

[britkat1980]

Thanks, that's working, but I get a different depreciation error:

(node:63) [DEP0066] DeprecationWarning: OutgoingMessage.prototype._headers is deprecated
(Use node --trace-deprecation ... to show where the warning was created)

There's couple of tweaks needed to work with GivTCP v3, but I'll create a PR back with that shortly.

[willholley]

Is there any need to use node at runtime at all? If nginx can be used to serve the static site instead that removes the need to have node/npm in the runtime image completely and knocks a few hundred MB off the size.

I knocked up a quick prototype locally - happy to PR if there's interest.

[britkat1980]

Always happy to slim down the image size. Send a PR over on the GivTCP repo

[DanielGallo]

The dashboard just needs something simple to render the static web content. @willholley your approach sounds good. Thanks!

[willholley]

Great - I filed a PR at britkat1980/giv_tcp#283