iFargle / headscale-webui

A simple Headscale web UI for small-scale deployments.
Other
619 stars 56 forks source link

fix: update machine references to node #127

Open l00ps opened 7 months ago

l00ps commented 7 months ago

This PR fixes #128

davidfreina commented 5 months ago

I think you missed one replacement of 'machine'. In the renderer.py on line 352 the machine page fail to load with a 502 if it enters this if-clause because the app.logger.debug statement still contains a reference to routes['machine'] which should be replaced with routes['node']. I have opened a PR to your branch to fix this last reference (l00ps/headscale-webui#2).

ClashTheBunny commented 4 months ago

Would a more robust way of doing this support both the old version and the new version?