-
-
Notifications
You must be signed in to change notification settings - Fork 687
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
Most instances can't import subscriptions, only official, and one unofficial instances #2274
Comments
Works here at adminforge.de, YMMV I suppose. There's a chance something in your subscriptions could be causing issues (one particular channel or some such) which would explain why it works for me but not for you. |
Could you maybe share the subscriptions privately to me? I've never been able to reproduce this 🤔 |
I had the same problem. In fact, you have to select the right instance in the preferences. So if you import your preferences before the subscriptions, the instance is not the right one because it also imports the previously saved instance. |
I never understood why loading preferences seemed to load my subscriptions too, is that normal? I was setting it up on a friend's computer, imported my preferences, and all of my subscriptions loaded on a browser with just the preferences.json and no login included... This issue was reproducible with one random subscription per json (tested with 5 different subscriptions.json, with and with and without logging in. Sadly, yt.777.fi is closing down in 3 days, it was the only other instance and kavin libre that had no cdn and could load subs without account. normal kavin.rocks doesn't have this problem when not logged in and no cookies/site data saved also. I'll retry this with other browsers. |
It's not an issue but a feature, I think ^^ |
This is different than the issue above. Please open a new issue!
It's an unexpected bug, preferences shouldn't really include subscriptions! |
@pinbuck is there any way for anyone to reproduce this issue? Feed loading works with my subscriptions on all instances I tested. However, I did notice that the video uploaded time was quite varying in different instances, which does sound worrying and I need to investigate that further. |
If it's a bug, what's the token for? |
That shouldn't be included either... |
We're currently making a backup of the whole local storage, and as local subscriptions are stored in the local storage and not in IndexedDb or server side they get backed up to. Though I don't see a reason to remove it, as there's no real disadvantage of it? Otherwise you'd have to create two different backups doing the same thing. |
True, but it does feel like it's unexpected behavior? I could imagine someone losing all their subscriptions by accident, especially since there's no mention of it... Maybe we should have an option to enable/disable it or, include a warning. Regarding auth tokens, they just shouldn't be included at all imo, for security reasons. |
I've reproduced this issue when I wanted to change instances. I exported my subscriptions from the previous instance, created an account on the other instance, and tried importing my subscriptions. Subscription count stayed 0 even though my file is valid and contains 98 subscriptions. So I cleared cookies, closed my browser to wipe everything else, reopened and re-logged in to the new instance, and this time the import worked as expected. I don't know if it could be reliably reproduced just by following these steps though. OP's issue was without being logged in at all in the first place... |
Imports take some time to happen as they happen asynchronously behind the scenes. In your case, the import likely took a while and was still happening. (it depends on how many subscriptions you have) |
It's always been instantaneous, every other time that I've tried. |
To reclarify: the way I use piped is without an account, this saves me time and keeps my password and data off of the many instances I have to switch back and fourth to when the ones I use go down. This hasn't been a problem for me that much anymore since I started just using the two official piped.kavin.rocks for auth which don't go down too much. I would still suggest any piped devs or piped instance owners to look into going accountless. (no sign-in or sign-up) |
This suggests that it might be because you had a really large subscriptions list, we were previously still using a GET request for these with the channels in the query parameters. This was 'fixed' in f2679c4, to use the POST method for large subscription lists.
We do currently support a good out of the box experience for accountless users (every feature currently works regardless of if you use an account), though there are no plans to make it a first class feature. In the future, things like Watch History, Preferences, Channel Groups will be e2ee, and syncable with an account. Unauthenticated users actually cause more server stress and load than authenticated, due to the large queries that are necessary (that can't be avoided) — while the impact of these could be better mitigated, it requires a lot of engineering effort (which I don't have time for). #1046 would also be helpful (for both accounts, and accountless). |
Official Instance
Describe the bug
I had another user confirm that my test results should be repeatable, he uses google chrome on windows and my own linux install using firefox with the same exact results.
Here are my test results:
instances.txt
I also tested the actual websites of each unofficial instance such as https://piped.esmailelbob.xyz/ and found there was no difference and the problem still occurs.
To Reproduce
Expected behavior
Importing subscriptions should work without an account logged in.
Logs/Errors
N/A
Browser, and OS with Version.
Fedora 37 Linux, Firefox 111.0.1 latest version and packages up to date.
Windows 10, Google Chrome 111.0.5563.146/147, latest version
Additional context
No response
The text was updated successfully, but these errors were encountered: