MyFavMirrors/owntone-server
1
0
Fork 0
mirror of https://github.com/owntone/owntone-server.git synced 2025-02-27 05:19:18 -05:00
Code Issues Packages Projects Releases Wiki Activity

[spotify] Handle channel error (so that new AP connection is triggered)

Browse Source
This commit is contained in:
ejurgensen 2022-03-15 21:14:28 +01:00
parent 75f6f75da3
commit 48051f6364
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/inputs/librespot-c/src/connection.c
View File

@ -699,6 +699,18 @@ response_aes_key_error(uint8_t *payload, size_t payload_len, struct sp_session *
return SP_ERR_DECRYPTION; return SP_ERR_DECRYPTION;
} }
// AP in bad state may return a channel error after chunk request. In that case
// we error with NOCONNECTION, because that will make the main session handler
// (see response_cb) retry with another access point. An example of this issue
// is here https://github.com/librespot-org/librespot/issues/972
static enum sp_error
response_channel_error(uint8_t *payload, size_t payload_len, struct sp_session *session)
{
sp_errmsg = "The accces point returned a channel error";
return SP_ERR_NOCONNECTION;
}
static enum sp_error static enum sp_error
response_mercury_req(uint8_t *payload, size_t payload_len, struct sp_session *session) response_mercury_req(uint8_t *payload, size_t payload_len, struct sp_session *session)
{ {
@ -783,6 +795,9 @@ response_generic(uint8_t *msg, size_t msg_len, struct sp_session *session)
case CmdMercuryReq: case CmdMercuryReq:
ret = response_mercury_req(payload, payload_len, session); ret = response_mercury_req(payload, payload_len, session);
break; break;
case CmdChannelError:
ret = response_channel_error(payload, payload_len, session);
break;
case CmdLegacyWelcome: // 0 bytes, ignored by librespot case CmdLegacyWelcome: // 0 bytes, ignored by librespot
case CmdSecretBlock: // ignored by librespot case CmdSecretBlock: // ignored by librespot
case 0x50: // XML received after login, ignored by librespot case 0x50: // XML received after login, ignored by librespot