Fix: publishedText is missing but the video isn't live or upcoming#9321
Fix: publishedText is missing but the video isn't live or upcoming#9321Shadorc wants to merge 1 commit into
Conversation
|
Most of the time when that message appears it is an actual issue that needs to be looked into, but I guess you'll just have to rely on noticing the parsing issues yourself. In this case with those unwatchable ads for a paid subscription service it is valid for the published date to be missing because they are not YouTube videos (the more correct solution would probably be to get rid of those ads like we do with members only videos, but for the moment I guess removing some genuinely useful logging will have to do). |
|
I haven't found a way to distinguish these videos (the ones without If you think it’s better to keep the error message because it’s more important to get an error when a viewable video has a parsing issue than to have no errors for a video that can’t be viewed, I understand, and this MR can be closed. |
|
@PikachuEXE @efb4f5ff-1298-471a-8973-3d47447115dc Which would you prefer: Silent failure in situations where it is an actual problem or errors in situations where it isn't actually an error? |
|
It's probably fine to remove, for the rest of FreeTube we also go with "the users will inevitably complain when something breaks and even when it doesn't" approach, so using that here too is probably fine. |
|
If the error only appears in console, I guess it's fine to get a warning (instead of current error |
Pull Request Type
Related issue
closes #8085
Description
It is a valid state to have an empty
publishedTextwhen the video is not live/premiere/upcoming.A good example is the paramountplus channel when using an IP located in USA:
YouTube:

FreeTube:

Differentiating valid and unvalid cases does not seem possible. It does not break FreeTube UI and does not impact the user when a video is returned without
publishedText, so I suggest to remove the log.Testing