-
Notifications
You must be signed in to change notification settings - Fork 13
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
Show error message when credentials are invalid #363
Conversation
Thanks for your contribution @Rainer-Keller :) I think it is a good idea to make the addon more user friendly! Maybe we can also identify other case where a notification makes sense (in separate PRs; afterwards). Slightly off-topic: I thought about removing the credentials authentication in favor of the device authentication. In that case it will hopefully a bit more user friendly in future. |
For regular users, it is not possible to find out what is wrong without a message being shown.
The plugin has to be placed in the build directory of Kodi to be found.
I helped someone to setup Kodi on a Raspberry Pi. This was quite a struggle for various reasons. The other issues are probably not in your plugin.
Is there a reason for this? More off-topic: The used Wifi was sometimes unstable causing the whole UI to freeze in the buffering state forever. The original Waipu stick also had occasional buffering animations but was able to continue playing after a few seconds. Kodi was not. It showed only "stream stalled" in the log, nothing else. |
Storing credentials in the settings is error-prone. If you change your password on the website, using credential authentication requires to update password in plugin settings. Another reason: the credentials are not well protected in kodi (e.g. accessible via HTTP API). Not storing the credentials in plugin prevents the from theft and ensures that they also will not appear in logs.
I have not noticed that issue related to an unstable Wifi. But it is not related to our plugin. The plugin only handles authentication, grabs data and prepares the stream. Everthing else is handled by kodi. Especially that streaming/buffering is implemented in kodi and inputstream.adaptive. After the stream is started, our plugin is not repsonsible for playback. |
For regular users it is too complicated to investigate the JSON result from the HTTP request in the debug log.
Showing the message in the UI makes it more obvious.