FlowFuse / node-red-dashboard

https://dashboard.flowfuse.com
Apache License 2.0
206 stars 49 forks source link

Support internal HTTPS for WYSIWYG #1480

Closed Steve-Mcl closed 2 days ago

Steve-Mcl commented 3 days ago

closes #1473

Description

Related Issue(s)

Checklist

Labels

Steve-Mcl commented 2 days ago

after further consideration, I am closing this in favour of a new PR (incoming) change based off the wysiwyg work in #1469 which had to change axios promise chain to async await.

This is to both avoid having a merge conflict AND the fact I was not checking the right thing to determined a HTTPS scheme was required. In short, Node-RED listens on HTTPS when RED.settings.https is set to "something" not the presence of the flag requireHttps