Fix to find the correct download path without language suffixes at first run

[ghost]

No description provided.

[AsciiWolf]

Thanks, looks good! I will however wait for other reviews (preferably from @micahflee or other contributors) before adding this patch to the Tor Browser Launcher Flatpak. :-)

edit: After this gets properly reviewed, it should be merged ASAP (probably along with #642) and new version of torbrowser-launcher should be released.

[copiummax]

lol but isn't this now hardcoded to the new resource? you did the same thing and looks like it will break again when it changes.

unless i'm wrong, the first endpoint GET /torbrowser/update_3/release/Linux_x86_64-gcc3/x/en-US HTTP/1.1\r\n replied with a correct resource to fetch and i think you should parse that somehow, otherwise what's the use of querying it in the first place

[AsciiWolf]

Good point.

[copiummax]

would still work as a hotfix, and it's important because the thing is quite completely broken for any new users. So depends on what you decide to do, i might work on proper parsing later if no one does

[ghost]

That's right. I installed this package for the first time today, found an error and uploaded the corrected one, so this hotfix was my best. I think I won't fix this problem completely. I'll appreciate it if you can work on it

[hackerncoder]

lol but isn't this now hardcoded to the new resource? you did the same thing and looks like it will break again when it changes.

unless i'm wrong, the first endpoint GET /torbrowser/update_3/release/Linux_x86_64-gcc3/x/en-US HTTP/1.1\r\n replied with a correct resource to fetch and i think you should parse that somehow, otherwise what's the use of querying it in the first place

Nope, it doesn't. It gives a link to a mar file. And you can't just s/mar/tar.xz/, that 404s

Edit: Ok, you can pull the file name out of that, and then splice together a correct link at dist.tpo.

[mavasbel]

Hotfix works

[nullsurface]

Sweet, hopefully this can get approved quick.

[soullessschizo]

thanks

[AsciiWolf]

There's also #660 that also removes the now unused code. Anyway, please see my comment. It will need a proper fix, preferably from (or approved by) @micahflee.

[micahflee]

Merged in #661