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

serve cover art that is embedded into audio file #15

Merged

Conversation

mgoltzsche
Copy link
Contributor

@mgoltzsche mgoltzsche commented Mar 25, 2024

Make the getCoverArt endpoint extract and serve the coverart image from the audio file for items that don't specify an album_id or refer to an album that does not specify an artpath.
Correspondingly, make the getSong endpoint return the song ID as coverArt ID when the song does not specify an album_id.

Extracts the cover art into a temporary file using ffmpeg and lets flask serve it.
Also, moves the getCoverArt endpoint implementation from album.py into a separate file since it is used to serve both album and single song cover art now.

@mgoltzsche mgoltzsche force-pushed the extract-coverart-from-item branch from 5d33bb8 to 1396c32 Compare March 25, 2024 00:53
@mgoltzsche mgoltzsche changed the title feat: serve cover art that is embedded into audio file serve cover art that is embedded into audio file Mar 25, 2024
@mgoltzsche mgoltzsche force-pushed the extract-coverart-from-item branch from 1396c32 to 0234238 Compare March 27, 2024 02:26
@mgoltzsche mgoltzsche marked this pull request as draft April 10, 2024 00:33
@mgoltzsche mgoltzsche force-pushed the extract-coverart-from-item branch from 0234238 to cc4715e Compare April 10, 2024 00:35
Make the `getCoverArt` endpoint extract and serve the coverart image from the audio file for items that don't specify an `album_id` or refer to an album that does not specify an `artpath`.
Correspondingly, make the `getSong` endpoint return the song ID as `coverArt` ID when the song does not specify an `album_id`.

Extracts the cover art into a temporary file using `ffmpeg` and lets flask serve it.
Also, moves the `getCoverArt` endpoint implementation from `album.py` into a separate file since it is used to serve both album and single song cover art now.
@mgoltzsche mgoltzsche force-pushed the extract-coverart-from-item branch from cc4715e to 8a3b05f Compare April 10, 2024 00:44
@mgoltzsche mgoltzsche marked this pull request as ready for review April 10, 2024 01:09
@BinaryBrain BinaryBrain merged commit a3177d2 into BinaryBrain:main May 7, 2024
@BinaryBrain
Copy link
Owner

Thanks a lot and sorry for the delay. I released v1.4.0.

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

Successfully merging this pull request may close these issues.

2 participants