schreibfaul1 / ESP32-vs1053_ext

With this library You can easily build a WebRadio with a ESP32 board and a mp3-module. See: https://www.youtube.com/watch?v=u4u9NvZvWRk
GNU General Public License v3.0
106 stars 29 forks source link

AUDIO_info: Metadata block to long! Skipping all Metadata from now on. Causes a sound problem: BR Radio only! #32

Closed TubaFonse closed 1 year ago

TubaFonse commented 1 year ago

Hello Wolle, due to your great prepared project and good documentation now I successfully built my 1st own MiniWebRadio V2 (ESP32-DevKitC & ext. VS1053-Module). But since a few days I got continuous sound distortions (bad "plopp's" and "quieks" from the speakers), but only when I listen any of the 14 different livestreams of "BR Radio (Bayrischer Rundfunk)". e.g. url's: https://streams.br.de/bayern1obb_2.m3u I found the reason of the problem via PlatformIO Serial Monitor:

AUDIO_info: Metadata block to long! Skipping all Metadata from now on.

I found, that all the stations of BR Radio send (~since a few days?) a longer Metadata-block, than expected/allowed in your library vs1053_ext.cpp (see line No. 3268):

if(!metalen) { int b = _client->read(); // First byte of metadata? metalen = b * 16 ; // New count for metadata including length byte if(metalen > 512){ AUDIO_INFO("Metadata block to long! Skipping all Metadata from now on."); m_f_metadata = false; // expect stream without metadata return 1; }

To test/try, how much more Metadata Bytes than 512 sends BR Radio from now on, I increased the value 512 + 16, then the sound distortions were finished!!! But I got a txt overflow ... If you can reproduce / confirm the problem, please help me to fix the problem "professional" by extend the length of Metadata block (Due to I'm a very bad programmer = beginner).

Many thanks, da TubaFonse.

TubaFonse commented 1 year ago

Problem is fixed in the actual revision. Comments see ESP32-MiniWebRadio, Issue #137