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.

Sign up for GitHub

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

Jump to bottom

"Media URL (Invalid)" error on some URLs #340

Closed
hellpe opened this issue Jun 9, 2023 · 10 comments · Fixed by #330
Closed

"Media URL (Invalid)" error on some URLs #340

hellpe opened this issue Jun 9, 2023 · 10 comments · Fixed by #330
Labels
bug Something isn't working in-beta This issue is fixed in a beta version
Milestone
V2023.6.1

Comments

@hellpe
Copy link

hellpe commented Jun 9, 2023 •
edited
Loading

Using the Flathub version on Fedora Silverblue 38

Although YouTube addresses work fine, URLs from ina.fr seem to have stopped being recognized as valid. Feeding those to ytp-dl from the command line does work though.

$ flatpak --env=LANG=en_US run org.nickvision.tubeconverter
Python.Runtime.PythonException: 'formats'
   at Python.Runtime.PythonException.ThrowLastAsClrException()
   at Python.Runtime.PyObject.GetItem(PyObject key)
   at Python.Runtime.PyObject.GetItem(String key)
   at Python.Runtime.PyObject.get_Item(String key)
   at NickvisionTubeConverter.Shared.Models.MediaUrlInfo.ParseFromPyDict(PyDict mediaInfo, Boolean isPartOfPlaylist) in /run/build/org.nickvision.tubeconverter/NickvisionTubeConverter.Shared/Models/MediaUrlInfo.cs:line 112
   at NickvisionTubeConverter.Shared.Models.MediaUrlInfo.<>c__DisplayClass13_0.<GetAsync>b__0() in /run/build/org.nickvision.tubeconverter/NickvisionTubeConverter.Shared/Models/MediaUrlInfo.cs:line 82

image
@nlogozzo
Copy link
Member

nlogozzo commented Jun 9, 2023 •
edited
Loading

This was actually fixed with #335 .

Could you provide me with the URL you used so I can double check?

@nlogozzo nlogozzo added this to the V2023.6.1 milestone Jun 9, 2023
@nlogozzo nlogozzo added the bug Something isn't working label Jun 9, 2023
@hellpe
Copy link
Author

hellpe commented Jun 9, 2023

Any URL from ina.fr is affected on my setup. Here's one: https://www.ina.fr/ina-eclaire-actu/video/cab92045133/dassault

nlogozzo added a commit that referenced this issue Jun 9, 2023
@nlogozzo
Fix Validation Of Some Sites
bd081b0 
#340
@nlogozzo nlogozzo mentioned this issue Jun 9, 2023
Merged
@nlogozzo
Copy link
Member

nlogozzo commented Jun 9, 2023

Got it fixed, will be included in the next release :)
image

nlogozzo added a commit that referenced this issue Jun 10, 2023
@nlogozzo @fsobolev
Add Keyring Feature + Embed Chapters + Fix Validation (#330)
42dc6be 
* Add Nickvision.Keyring

* GNOME - Add Keyring Action

* Add Blank Keyring Dialog + Controller

* Update Nickvision.Keyring

* GNOME - (WIP) Keyring Toggle

* Update KeyringDialog.cs

* Update keyring_dialog.blp

* Destroy Keyring In Dialog

* Enable Keyring In Dialog

* Update MainWindow Keyring After Dialog Close

* Unlock Keyring At Startup

* Better Disable Keyring Message

* Even Better Disable Keyring Message

* Better NewPasswordDialog

* Empty Keyring Design

* Account -> Credential

* GNOME - (WIP) Credential Page

* Embed Chapters Support

* GNOME - KeyringDialog Add Button Handlers

* KeyringDialog Validate and Add Credential

* Finish KeyringDialog

* Update tubeconverter.pot

* GNOME - Show spinner when loading credentials

* Update KeyringDialog.cs

* Fix Duplicate Credentials in KeyringDialog

* Fix Validation Of Some Sites

#340

* Update tubeconverter.pot

* Finish AddDownloadDialog Keyring Support

* Update tubeconverter.pot

* GNOME - Add Auth Row Switch

* Update tubeconverter.pot

* GNOME - Correctly disable keyring

* No Magic Numbers

---------

Co-authored-by: Fyodor Sobolev <[email protected]>
@nlogozzo
Copy link
Member

https://github.com/NickvisionApps/TubeConverter/releases/tag/2023.6.1-beta2

@nlogozzo nlogozzo added the in-beta This issue is fixed in a beta version label Jun 10, 2023
@taiwbi
Copy link

taiwbi commented Jul 22, 2023

I still get this error on both 2023.7.3-beta and 2023.7.2 installed from falthub :(

Screenshot from 2023-07-22 10-18-05
Screenshot from 2023-07-22 10-18-17

@fsobolev
Copy link
Member

@taiwbi any output when running in terminal?

@nlogozzo
Copy link
Member

@taiwbi can you paste the link here so we can investigate it?

@fsobolev
Copy link
Member

@nlogozzo https://www.youtube.com/watch?v=e74l3uPQLYc Works fine for me.

@nlogozzo
Copy link
Member

nlogozzo commented Jul 22, 2023 •
edited
Loading

Yeah works for me too in current stable:
image

@taiwbi
Copy link

taiwbi commented Jul 22, 2023

When I run it from terminal it works. The problem was that Youtube is blocked in my region and when I open application from apps grid, it doesn't follow system proxy settings.

I thought about it when I comment at first so I tried to download the video with VPN mode on nekoray which supposed to force proxy all the traffic but it didn't worked with that too :/

I fixed it by adding proxy variables for Parabolic on Flatseal.

all_proxy
HTTPS_PROXY
ALL_PROXY
https_proxy
HTTP_PROXY
http_proxy
no_proxy
NO_PROXY
ftp_proxy
FTP_PROXY

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working in-beta This issue is fixed in a beta version
Projects
None yet
Milestone
V2023.6.1
Development

Successfully merging a pull request may close this issue.

Add Keyring Feature + Embed Chapters + Fix Validation NickvisionApps/Parabolic
4 participants
@hellpe @nlogozzo @taiwbi @fsobolev