-
Notifications
You must be signed in to change notification settings - Fork 444
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
ValueError: Can redirect only to http or https #7801
Comments
This comment was marked as duplicate.
This comment was marked as duplicate.
The following actions are performed:
|
@qstokkink, what do you think about the safety of allowing redirects in Tribler? I can either: |
I consider the existence of this issue to be evidence that redirection is a feature that our users want. Based on that, my vote goes to a) allow redirects. |
The error occurred here: if scheme not in ("http", "https", ""):
resp.close()
raise ValueError("Can redirect only to http or https") This is because, in our case, the scheme is Refs: |
I've implemented the desired feature, but it turns out that the magnet link shortened by TinyURL cannot be parsed by our |
Sentry Issue: TRIBLER-13Z
Last Seen
3 days ago
in release 7.13.1
First Seen
10 months ago
in release 7.12.1
EVENTS
11
USERS
3
The text was updated successfully, but these errors were encountered: