~lomanic


#42 Properly handle non-existing channels 1 year, 6 months ago

Comment by ~lomanic on ~cadence/tube

This issue can be closed, there's no stacktrace returned anymore by CloudTube for non-existing channels.

#29 Support alternative channel URLs 1 year, 6 months ago

Comment by ~lomanic on ~cadence/tube

This shouldn't have been closed as Implemented, as the MR or commit said, this is only partially fixed as /{name} channels are still not properly displayed (Cloudtube may redirect to /watch?={name} if {name} matches YouTube video ID format).

In the meantime, a new /@{name} format appeared, such as https://www.youtube.com/@YouTube (name is case-independent for this format also).

I have implemented support for the /@{name} format in https://git.sr.ht/~lomanic/cloudtube/commit/1b13680c5a6f0d5ca9d2218c799e5ce55ab9bf53 and https://git.sr.ht/~lomanic/NewLeaf/commit/e601f21dd696f91c4adbf2f450c470c318ac3e06, in the alternative-channel-names-at and proxy-videos branches respectively. If you are interested in pulling these changes.

#45 Discard post overlay with the Escape key 3 years ago

Ticket created by ~lomanic on ~cadence/bibliogram-issues

When accessing a profile page and clicking on a post image, an overlay is superimposed over the grayed page to display the post (the page is not refreshed). To discard the post overlay, you have to click on the gray part of the page, outside the post overlay.

While there are shortcuts to navigate posts with ← →, I often find myself trying to close the overlay by pressing the Escape key like on many websites, without success. I would appreciate if pressing Escape could discard the post overlay.

#29 Support alternative channel URLs 3 years ago

Comment by ~lomanic on ~cadence/tube

Are there any others?

https://github.com/yt-dlp/yt-dlp/blob/0fa9a1e23625f0ba4516e5107ce447ac693e7ec1/yt_dlp/extractor/youtube.py#L2467

I think there's no more routes.

if someone has /c/stage, does this also automatically give them /user/stage and /stage?

https://www.youtube.com/c/AndreasSpiess redirects (this is new: "short links" redirect to the "slug" one now, this is done client-side and it looks like it's only a call to history.replaceState()) to https://www.youtube.com/channel/UCu7_D0o48KbfhpEohoP7YSQ, while https://www.youtube.com/user/AndreasSpiess is some another Avocado empty user, redirected to https://www.youtube.com/channel/UCVRacX7DnkMLNrQPQWvgNZA

https://www.youtube.com/AndreasSpiess redirects properly to https://www.youtube.com/channel/UCu7_D0o48KbfhpEohoP7YSQ

https://tube.cadence.moe/channel/AndreasSpiess (https://redirect.invidious.io/channel/AndreasSpiess also) gives this Avocado user. https://www.youtube.com/channel/AndreasSpiess is a 404.

Is it acceptable to redirect from that name to the standard /channel/{identifier}, or must I render the response directly on those routes?

YouTube now rewrites the URL to the slug one (amusingly, if you click on any tab like Videos, the original visited old short link shows up), we can follow that if it makes it simpler on CloudTube part.

#42 Properly handle non-existing channels 3 years ago

enhancement added by ~lomanic on ~cadence/tube

#42 Properly handle non-existing channels 3 years ago

Ticket created by ~lomanic on ~cadence/tube

Currently, this deleted channel returns a stackstrace on NewLeaf https://www.youtube.com/channel/UC6v6ljwkDGjk8Q7Dj-yjuqg

Traceback (most recent call last):
  File "/home/cloudtube/newleaf-venv/lib/python3.5/site-packages/cherrypy/_cprequest.py", line 638, in respond
    self._do_respond(path_info)
  File "/home/cloudtube/newleaf-venv/lib/python3.5/site-packages/cherrypy/_cprequest.py", line 697, in _do_respond
    response.body = self.handler()
  File "/home/cloudtube/newleaf-venv/lib/python3.5/site-packages/cherrypy/lib/encoding.py", line 219, in __call__
    self.body = self.oldhandler(*args, **kwargs)
  File "/home/cloudtube/newleaf-venv/lib/python3.5/site-packages/cherrypy/lib/jsontools.py", line 59, in json_handler
    value = cherrypy.serving.request._json_inner_handler(*args, **kwargs)
  File "/home/cloudtube/newleaf-venv/lib/python3.5/site-packages/cherrypy/_cpdispatch.py", line 54, in __call__
    return self.callable(*self.args, **self.kwargs)
  File "/home/cloudtube/NewLeaf/index.py", line 83, in channels
    return extract_channel(ucid)
  File "/home/cloudtube/NewLeaf/extractors/channel.py", line 26, in extract_channel
    channel_metadata = yt_initial_data["metadata"]["channelMetadataRenderer"]
KeyError: 'metadata'

This should be properly handled, and the same way as Invidious API.

The error code also needs to be handled in CloudTube.

#41 &t=startTime parameter broken on videos 3 years ago

Comment by ~lomanic on ~cadence/tube

#41 &t=startTime parameter broken on videos 3 years ago

Comment by ~lomanic on ~cadence/tube

(fixed links in OP)

There's no Best combined in the /settings page on my instance, so tested everything:

  • 720p: initial default, not working, starts at 00:00
  • 360p: not working, starts at 00:00
  • Best possible: worked once, but track time was locked on 3:15 while it was playing, after a refresh was not working anymore, starts at 00:00
  • Best <= 1080p: not working, starts at 00:00
  • Best <= 30fps: not working, starts at 00:00

#41 &t=startTime parameter broken on videos 3 years ago

problem added by ~lomanic on ~cadence/tube

#41 &t=startTime parameter broken on videos 3 years ago

cloudtube added by ~lomanic on ~cadence/tube