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

Can't get Soundcloud connection at the moment (status code 401) #71

Closed
mrvanes opened this issue Feb 1, 2016 · 3 comments
Closed

Can't get Soundcloud connection at the moment (status code 401) #71

mrvanes opened this issue Feb 1, 2016 · 3 comments
Labels
C-bug Category: This is a bug

Comments

@mrvanes
Copy link

mrvanes commented Feb 1, 2016

It seems Soundcloud returns a 401 on authentication request but the code isn't prepared to handle the answer (attribute response is missing). I'm quite sure the auth_token I use is correct.

ERROR    Got un-handled exception from SoundCloudBackend
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/mopidy/commands.py", line 239, in _actor_error_handling
    yield
  File "/usr/lib/python2.7/dist-packages/mopidy/commands.py", line 380, in start_backends
    config=config, audio=audio).proxy()
  File "/usr/lib/python2.7/dist-packages/pykka/actor.py", line 94, in start
    obj = cls(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/mopidy_soundcloud/actor.py", line 21, in __init__
    self.remote = SoundCloudClient(config['soundcloud'])
  File "/usr/lib/python2.7/dist-packages/mopidy_soundcloud/soundcloud.py", line 80, in __init__
    if err.response is not None and err.response.status_code == 401:
AttributeError: 'ConnectionError' object has no attribute 'response'
@jodal jodal added the C-bug Category: This is a bug label Feb 1, 2016
@mrvanes
Copy link
Author

mrvanes commented Feb 2, 2016

Seems fixed now, probably hickup @ Soundcloud side?

@jodal
Copy link
Member

jodal commented Feb 2, 2016

Probably, but leaving this open as we should probably replace err.response is not None with getattr(err, 'response', None) is not None or similar.

@kingosticks
Copy link
Member

The error handling is improved in #100, I believe this is now fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug
Projects
None yet
Development

No branches or pull requests

3 participants