Skip to content
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

Fix Emby notifier error on Python 3 #7497

Merged
merged 2 commits into from
Dec 15, 2019
Merged

Fix Emby notifier error on Python 3 #7497

merged 2 commits into from
Dec 15, 2019

Conversation

sharkykh
Copy link
Contributor

@sharkykh sharkykh commented Dec 15, 2019

From #7486

Traceback (most recent call last):
  File "/app/medusa/medusa/server/web/core/base.py", line 251, in async_call
    result = function(**kwargs)
  File "/app/medusa/medusa/server/web/home/handler.py", line 434, in testEMBY
    result = notifiers.emby_notifier.test_notify(unquote_plus(host), emby_apikey)
  File "/app/medusa/medusa/notifiers/emby.py", line 80, in test_notify
    return self._notify_emby('This is a test notification from Medusa', host, emby_apikey)
  File "/app/medusa/medusa/notifiers/emby.py", line 60, in _notify_emby
    log.debug('EMBY: HTTP response: {0}', resp.content.replace('\n', ''))
TypeError: a bytes-like object is required, not 'str'

Fixes #7339 too, it seems.

@sharkykh sharkykh added this to the 0.3.10 milestone Dec 15, 2019
@medariox medariox merged commit ddc5876 into develop Dec 15, 2019
@medariox medariox deleted the bugfix/emby-py3-error branch December 15, 2019 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants