mediathekview / zapp

German public broadcasting live streams as an Android app
https://mediathekview.de/news/zapp/
MIT License
209 stars 34 forks source link

Incorrect umlaut and `ß` display #450

Closed WebworkrNet closed 2 months ago

WebworkrNet commented 2 months ago

Steps to reproduce Call up a tab or a detailed view. Expected behaviour Proper character set representation Actual behaviour Umlauts and ß are displayed incorrectly. Device: Samsung Galaxy A13 5G (SM-A136B/DSN) Operating System: Android 14, One UI-Version 6.1 ZAPP: 8.5.2 (72)

WebworkrNet commented 2 months ago

As the phenomenon has only been occurring for a few days and the last app update was some time ago, the cause is probably to be found in upstream Mediathekview or ZAPP components or at the broadcasters.

I am trying to find out whether the problem can be narrowed down to certain channels.

WebworkrNet commented 2 months ago

ZDF

Screenshot_20240907_005615_Zapp Screenshot_20240908_190342_Zapp Screenshot_20240908_191524_Zapp

strelitzer commented 2 months ago

Same on Galaxy A72 and Galaxy Tab A8, both with Samsung One UI 6.1 (Samsung keyboard, different/installed font) and v8.5.2(72).

cemrich commented 2 months ago

Thanks for reporting!

The encoding of the mediathekviewweb api seems to be off for these shows. Only a few shows are affected and they look fine on mediathekviewweb.de

Maybe @bagbag can help debugging this?

bagbag commented 2 months ago

The cause was this bug: https://github.com/nodejs/node/issues/54543

Last update was made with that version. Update is deployed, so it should display correctly now.

WebworkrNet commented 2 months ago

Phoenix live stream

Screenshot_20240909_225309_Zapp

WebworkrNet commented 2 months ago

Will the change only affect new programs or also programs that have already been imported into the database by Mediathekview?

bagbag commented 2 months ago

I guess there were also some corrupted database entries. I've just triggered a full reindex, so older entries should be fine too.

cemrich commented 2 months ago

The cause was this bug: https://github.com/nodejs/node/issues/54543

Thanks you very much for the fix!

Phoenix live stream

This is a separate issue.

WebworkrNet commented 2 months ago

Phoenix live stream

This is a separate issue.

@cemrich Is there already a bug report on this?