radio-t / radio-t-site

Сайт подкаста Радио-Т
https://radio-t.com
MIT License
103 stars 59 forks source link

С 739 выпуска сломались главы (chapters) #209

Closed rakleed closed 3 years ago

rakleed commented 3 years ago

С 739 выпуска сломались главы (chapters) — в Apple Podcasts и Pocket Casts они не отображаются. А Overcast видимо сам делает главы с помощью timestamps из описания.

Скриншоты ![Image-1](https://user-images.githubusercontent.com/19418601/108692757-10655380-750e-11eb-8fe9-4873873cc91c.png) ![Image-3](https://user-images.githubusercontent.com/19418601/108692761-11968080-750e-11eb-9a34-b2acf2c55b53.png)

@q210 не могли бы вы, пожалуйста, посмотреть, в чём может быть проблема?

q210 commented 3 years ago

Добрый день, действительно что-то где-то отломалось, большое спасибо за сигнал.

Починю.

q210 commented 3 years ago

хм, в 743м эпизоде главы вновь показываются в Podcasts на маке и в iOS, при том что я ничего не делал еще (попытался воспроизвести проблему в понедельник, увидел что действительно нет глав в последних выпусках, оставил на выходные когда время будет).

При этом как в файле 742го эпизода, так и в файле 743го эпизода все главы есть в тэгах (проверил, скопировав файлы эпизодов из кэша Podcasts на маке), однако как видим эппловый софт отображает эти главы лишь для 743го.

Все страньше и страньше, пора призывать аксакалов.

@umputun подскажи плз - ничего не менялось в процессе релиза между 742 и 743 ? В коммитах ничего подозрительного не вижу.

Если в процессе релиза ничего не менялось, следующее подозрительное место - в тексте описания, который парсится скриптами для создания разметки глав, буду копать чем они отличаются в эпизодах где глав нет.

umputun commented 3 years ago

нет, я ничего не чинил. Там, судя по всему, что–то падает в зависимости от тем выпуска. Я нa это обращал вниманиe когда выкладывал в tg:

fy4gv-202102-28132455-vb2wa
q210 commented 3 years ago

ok, экспериментальным путем установил, что Apple не нравится используемая нами версия стандарта ID3 тэгов.

Мы используем ID3v2.4, от 2003го года (https://id3.org/id3v2.4.0-frames), и любые названия глав длиннее 51 юникодного символа ломают отображение глав в Podcasts на MacOS и iOS.

Переформатировал теже оглавления в формат ID3v2.3, от 1999го года (https://id3.org/id3v2.3.0) и ограничение на длинну пропало.

q210 commented 3 years ago

@umputun фикс из pull-request исключит повторение ситуации в будущем, но я не уверен что делать со старыми выпусками.

Откуда пользователи Podcasts выкачивают mp3 файлы прошлых эпизодов? С какого-то CDN? Мы можем перезалить туда файлы, предварительно пересохранив тэги в более старом формате?

Или слишком много возни, и хватит лишь фикса для будущих выпусков?

umputun commented 3 years ago

я могу перезалить, это небольшая проблем.

umputun commented 3 years ago

пересобрал и перезалил 742, вроде главы на месте. Не уверен как это просто проверить в podacst но quick view их теперь показывает и tg тоже

q210 commented 3 years ago

здорово! Для того чтобы перегенерировать тэги в нужном формате для старых эпизодов надо выполнить команду вида docker-compose run --rm publisher set-mp3-tags rt_podcast<number>/rt_podcast<number>.mp3, и перезалить

пример: docker-compose run --rm publisher set-mp3-tags rt_podcast685/rt_podcast685.mp3

umputun commented 3 years ago

для чего?

q210 commented 3 years ago

сорри, рано отправился комментарий, апдейтнул

umputun commented 3 years ago

я имею ввиду, что у меня общее flow которым проще все одним ударом сделать, оно и таги поставит и зальет на все узлы. Запустил для всех до 740 ....

umputun commented 3 years ago

все (до 739) перестроил и залил

q210 commented 3 years ago

@Rakleed у меня в Podcasts, после удаления эпизодов до и после 739го и скачивания их заново, главы стали видны, проверь плз тоже

rakleed commented 3 years ago

У меня с 739 по 742 выпуск и в Apple Подкастах, и в Pocket Casts отображаются главы. @q210 и @umputun, спасибо за фикс!