-
Notifications
You must be signed in to change notification settings - Fork 1
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
Android notifier error #22
Comments
Please install v2.0.1 from https://github.com/BobVul/GrowlToToast/releases/tag/v2.0.1 I've updated the logging to provide more information (and actually write the file!). You can find the logs under If you could provide those logs, that'd be great. I have a vague idea how to fix this (either strip out or encode any random null characters) but I'd like to know which field they're appearing in. |
Oh, you'll also need to enable debug logging via the Growl settings panel (new checkbox). That will provide far more detail than just the error logging. |
|
Ok, there it is. That is incredibly dodgy (it stinks of an off-by-one error somewhere in C code, probably in Growl or Android Notifier, because nulls terminate C strings), but I suppose I should be handling them just in case. |
@trentyarwood Try v2.0.2 https://github.com/BobVul/GrowlToToast/releases/tag/v2.0.2 Don't forget to turn off debug logging if it works. It tends to flood the logs. |
Success! |
\o/ |
GtT 2.0
Windows 10 Pro v 1803
Growl 2.0.9.1
MultiDroid notifier 0.51_x64 - https://code.google.com/archive/p/android-notifier/downloads\
Windows Java v 1.8.0_171
Android P preview on Google Pixel
Remote notifier 0.2.8
PC > Growl > Display > GTT > Preview > works as expected
Android > Remote notifier > Send test notification > works as expected when display set to (notGTT)
Android > Remote notifier > Send test > Error (see screen dump below, but no log file created at the file location in the first line)
The text was updated successfully, but these errors were encountered: