Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Update plugin.py #114

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TehPeGaSuS
Copy link

@TehPeGaSuS TehPeGaSuS commented Jan 26, 2019

Changed the plugin to work with Icecast 2.4.x (tested with Icecast 2.4.4).

  • Line 107 - Removed the [0] due to JSON output changes.
  • Line 117 and 141- Replaced inexistent artist key in the actual JSON format by server_name key (the actual DJ).

Thanks to cottongin and pinkieval for the help in #limnoria @ Freenode

Changed the plugin to work with Icecast 2.4.x (tested with Icecast 2.4.4).
1) Line 107 - Removed the `[0]` due to JSON output changes.
2) Line 117 and 141 replaced inexistent `artist` key in the actual JSON format by `server_name` key (the actual DJ).
We should rely only on track name for track changes announcement since we have no `artist` key and `server_name` can be streaming for hours
@TehPeGaSuS
Copy link
Author

@butterscotchstallion could you please the pending PRs? would be great to people having something working with newer icecast versions tho.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant