Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Thumbnail links are sometimes saved to playlists.db #5601

Closed
5 of 6 tasks
TheodorSmall opened this issue Aug 25, 2024 · 4 comments · Fixed by #5900
Closed
5 of 6 tasks

[Bug]: Thumbnail links are sometimes saved to playlists.db #5601

TheodorSmall opened this issue Aug 25, 2024 · 4 comments · Fixed by #5900

Comments

@TheodorSmall
Copy link
Contributor

TheodorSmall commented Aug 25, 2024

Guidelines

  • I have encountered this bug in the latest release of FreeTube.
  • I have encountered this bug in the official downloads of FreeTube.
  • I have searched the issue tracker for open and closed issues that are similar to the bug report I want to file, without success.
  • I have searched the documentation for information that matches the description of the bug I want to file, without success.
  • This issue contains only one bug.

Describe the bug

  1. Use invidious API (preferably disable "fallback to secondary API") and select instance 1
  2. Save a playlist from YT
  3. Optionally change the invidious instance to instance 2 and assume instance 1 is no longer working
  4. In playlists.db, you will find that there are some attributes to the videos of a playlist that are never used.

These attributes would not be saved when adding the videos one by one to the playlist:

  • videoThumbnails
  • authorUrl
  • liveNow
  • index

Expected Behavior

The attributes should be removed when saving the playlist.

Issue Labels

inconsistent behavior

FreeTube Version

Every version since playlists could be copied and saved locally.

Operating System Version

Artix Linux

Installation Method

.pacman, built from source

Primary API used

Invidious API

Last Known Working FreeTube Version (If Any)

unknown, probably none

Additional Information

No response

Nightly Build

@TheodorSmall

This comment was marked as outdated.

@efb4f5ff-1298-471a-8973-3d47447115dc

I cant seem to find the identifier for the thumbnail links that are listed in the playlists.db. Are you sure you are talking about the right file?

Copy link
Contributor

This issue has been automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take action. Please reach out if you have or find the answers we need so that we can investigate further.

@TheodorSmall
Copy link
Contributor Author

In fact, playlists.db is the right file, and although the video thumbnails should not be saved there, they are saved with the playlist, however only when adding multiple videos at once. The fix is quite easy because there is already a list of "undesired attributes" in the addVideos function in src/renderer/store/modules/playlists.js.
There are also a few other attributes that are only saved when adding multiple videos. These are all additional attributes that are not yet contained in the list:

  • videoThumbnails
  • authorUrl
  • liveNow
  • index

This is not a privacy issue, since the links in question are never read.

@TheodorSmall TheodorSmall changed the title [Bug]: Thumbnail links in playlists.db refer to the invidious instance used when adding the video, not to YT [Bug]: Thumbnail links are sometimes saved to playlists.db Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants