Ylianst / MeshCentral

A complete web-based remote monitoring and management web site. Once setup you can install agents and perform remote desktop session to devices on the local network or over the Internet.
https://meshcentral.com
Apache License 2.0
3.67k stars 511 forks source link

Battery Percentage no longer showing on many Laptop Types (since 1.1.23) #6143

Closed NilsLeumann closed 1 month ago

NilsLeumann commented 1 month ago

Describe the bug Since version 1.1.23 and also on version 1.1.24 many laptops that used to show the battery state of charge now no longer show it (no battery Icon, just like the computer doesn't have a battery) Tested Laptops: Surface Pro 6 (only one affected), ThinkPads (but only a P16, all my E14 AMD show battery just fine), HP ProBooks (only Intel models, so far no AMD) In my testing it mainly seemed to affect intel based laptops, however I have not yet been able to determine a clear pattern

To Reproduce Steps to reproduce the behavior: Well, just log on to meshcentral and check if there is a battery icon :)

Screenshots If applicable, add screenshots to help explain your problem.

Server Software (please complete the following information):

Client Device (please complete the following information):

Remote Device (please complete the following information):

Bildschirmfoto 2024-06-01 um 22 16 47
si458 commented 1 month ago

yeh ive spotted this recently too and i cant work it out whats happened? i moved the power-monitor.js from meshagent to meshcentral so it fixed the battery tracking for mac laptops https://github.com/Ylianst/MeshCentral/commit/b71b4d04e67175492f608c3b69eb5b2ce2b22804 but it seems to of possibly broke the windows battery?

can you try one thing for me, delete agents/modules_meshcore/power-monitor.js then restart ur meshcentral server?

does the battery levels etc return?

si458 commented 1 month ago

yes deleting node_modules/meshcentral/agents/modules_meshcore/power-monitor.js and restarting meshcentral restored the battery levels for laptops, it just loses the battery levels for mac again so i will do more investigating again

NilsLeumann commented 1 month ago

Thank you very much for your quick reply, since I do not manage any Macs in MeshCentral I deleted the power-monitor.js or rather renamed it and added a .old Now it seems to work fine, at least on one of my HP Laptops that didn't show the battery percentage before. I assume all other devices will be fine now but I will see on monday I also just want to thank you for making and maintaining such an amazing piece of open source software :))

si458 commented 1 month ago

@NilsLeumann glad it temporarily sorted the issue. im gunna look into it fully tho as i am confused? as you said it seemt to show on some but then not others and i dont know how i didnt spot it! glad u like the software! dont forget to donate ❤️ https://www.si458.co.uk/2024/01/05/donation/

si458 commented 1 month ago

closing this as i have removed the file from the master image and npm install Ylianst/MeshCentral