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

Youtu.be domain+url rewriter #56

Closed
unquote7083 opened this issue Aug 4, 2022 · 4 comments · Fixed by #65
Closed

Youtu.be domain+url rewriter #56

unquote7083 opened this issue Aug 4, 2022 · 4 comments · Fixed by #65
Assignees
Labels
feature New feature or request

Comments

@unquote7083
Copy link

unquote7083 commented Aug 4, 2022

https://youtu.be/video-id should be transformed into https://www.youtube.com/watch?v=video-id. These youtu.be links comes from the share function.

Thanks for the app!

@svenjacobs
Copy link
Owner

svenjacobs commented Aug 4, 2022

Thanks for your feedback. Usually Leon tries to remove obsolete parameters and reduce the size of URLs. There is one exception to this rule which is the Amazon Smile rewriter. But in this particular case the benefit of rewriting the URL is to collect donations for charities.

Why do you want to convert a shorter youtu.be link to a longer youtube.com one?

@svenjacobs svenjacobs added the question Further information is requested label Aug 4, 2022
@unquote7083
Copy link
Author

The short url is specific to the share function and helps google track such. It redirects to
https://www.youtube.com/watch?v=video-id&feature=youtu.be and thus should be sanitized before it can get to that point

@svenjacobs
Copy link
Owner

Okay, good point. I will consider this.

@svenjacobs svenjacobs self-assigned this Aug 8, 2022
@svenjacobs svenjacobs added feature New feature or request and removed question Further information is requested labels Aug 8, 2022
@svenjacobs
Copy link
Owner

This will be available in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants