-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WinNUT immediately disconnects upon UPS going on battery power #5
Comments
Copy from the previously mentioned issue I have not tested this, but you can possibly skip 1 to 4 and use the "latest" version and do the same thing
After following these steps it worked fine for me, however just installing the newest pre release version (https://github.com/gawindx/WinNUT-Client/releases/tag/2.1.7740.35837) at the time of writing did not fix the issue even though it said so in the changelog EDIT: I should add that if you restart WinNUT through the shortcut on your desktop that the installer puts there it will just add the dll file back |
Thank you for making this issue report. I also appreciate that you included steps to reproduce the issue, but I'm not sure they are the same? I think the gawindx/WinNUT-Client#140 issue you linked above is also related to gawindx/WinNUT-Client#133, but the steps you mentioned in your second comment sound like gawindx/WinNUT-Client#74, gawindx/WinNUT-Client#135, and #3 here on the repository. Can you confirm if these are indeed separate issues? |
The steps i posted are not steps to reproduce the issue, they fix the issue As far as I can tell these are all the same issue, if you look through the log file i attached in the issue on the no 140 issue you find this: which is the same as in numbers 3 135 74 and i suspect quite a few others with similar problems |
Another thing: WinNUT Bug Report WinNUT Parameters : Error Message : Last Events : |
Ok, thank you for posting and clearing that up for me. I'm working on a lot of changes in #11 at the moment for another issue, but between that work and the new German translations, maybe this problem will be fixed. Please stay tuned for a pre-release (make sure you're watching the project) and I'll try to have this issue fixed. For my own reference, here's what the error reads in English: Exception type: System.ArgumentOutOfRangeException |
@MartinKurtz I've just made a pre-release here. This issue wasn't the focus of the build, but just in case I'd like you to check and see if the problem still exists. If so, then please make sure you upload the full debug log here. |
Simply downloading the archive and running it yields this error report: WinNUT Bug Report WinNUT Parameters : Error Message : Last Events : |
Removing the dll file from the de-DE-folder fixes the issue and allows me to simply run the archive version, which works, at least for now |
If NotifyStr.Length > 64 Then As far as i can tell the issue is that if you get a number between 59 and 65 the length argument you get becomes 0 or negative, which causes this exeption, so changing this to If NotifyStr.Length > 59 Then "should" fix at least this exeption, but i am not sure if this will cause issues somewhere else in the program |
I would like to compile the project myself, however i do not have any experience with VS, what additions to VS do i need to run and compile this project |
- Fixes NRE when closing form before creating UPS object - Clamp NotifyIcon text length to 63 characters or under (should address issue #5)
Looks like my first release didn't go so well! 😅 I've just made a hotfix release here, please download your preferred format again and let me know how it goes.
I looked at the offending line in the code. So from the documentation, the Text field must be 63 characters or less. I don't know what the
Thank you for your interest in contributing! I made a small info section in the README here, I'm curious if you'd like me to extend or add to it to make it more informative. Basically, you must have Visual Studio 2019 (community is fine) to compile the project. It depends on .Net Framework 4.7.2 at the latest, so we can't use Studio 2022 yet. If you want to compile the installer too, there's a link to what you need for that. Thank you again for your help with fixing these bugs! |
Okay, the setup.exe ran through perfectly, installed correctly. The string length issue also does not occur any longer for me at least |
Since the original Issue has been fixed, do you want to move this conversation about the readme somewhere else? |
Let's do that, thank you again for your feedback and help. I'll close this issue as fixed, then feel free to open another issue regarding the readme. |
See here
gawindx/WinNUT-Client#140
The text was updated successfully, but these errors were encountered: