Skip to content

Fix: publishedText is missing but the video isn't live or upcoming#9321

Open
Shadorc wants to merge 1 commit into
FreeTubeApp:developmentfrom
Shadorc:fix/published-text-error
Open

Fix: publishedText is missing but the video isn't live or upcoming#9321
Shadorc wants to merge 1 commit into
FreeTubeApp:developmentfrom
Shadorc:fix/published-text-error

Conversation

@Shadorc

@Shadorc Shadorc commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

closes #8085

Description

It is a valid state to have an empty publishedText when the video is not live/premiere/upcoming.

A good example is the paramountplus channel when using an IP located in USA:

YouTube:
image

FreeTube:
image

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

  1. Go to paramountplus channel using an IP located in USA
  2. Check that there is no error

@github-actions github-actions Bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 23, 2026
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) June 23, 2026 23:22
@absidue

absidue commented Jun 24, 2026

Copy link
Copy Markdown
Member

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).

@Shadorc

Shadorc commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

I haven't found a way to distinguish these videos (the ones without publishText) from the others, but maybe there is one.
I will try searching again to see if I find something.

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.

@absidue

absidue commented Jun 24, 2026

Copy link
Copy Markdown
Member

@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?

@absidue

absidue commented Jun 24, 2026

Copy link
Copy Markdown
Member

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.

@PikachuEXE

Copy link
Copy Markdown
Member

If the error only appears in console, I guess it's fine to get a warning (instead of current error
No idea what's the ratio of valid vs actual issue current date, but I guess if it's warning in console only normal user won't see it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: waiting for review For PRs that are complete, tested, and ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Posts tab throws silent error publishedText is missing but the video isn't live or upcoming

3 participants