Skip to content
This repository has been archived by the owner on Jun 26, 2022. It is now read-only.

No accentuated characters #4

Closed
thiswillbeyourgithub opened this issue Nov 19, 2018 · 8 comments
Closed

No accentuated characters #4

thiswillbeyourgithub opened this issue Nov 19, 2018 · 8 comments

Comments

@thiswillbeyourgithub
Copy link

thiswillbeyourgithub commented Nov 19, 2018

Hi,

I don't know if it's the right place to put this but I think I found a bug.

If on cloudtube I look for "test" it works fine but for "tést" (url : https://cadence.gq/cloudtube/search?q=t%C3%A9st&order=relevance )then I get Error requesting data from Invidious.

Given that invidious returns results for "tést" (url : https://invidio.us/search?q=t%C3%A9st) I think the problem is on your side.

Thanks for reading this :)

@cloudrac3r
Copy link
Owner

Did some reading and it's possibly a bug in node.js. nodejs/node#13296

The response from the HTTP server says the requested path is /cloudtube/search?q=t��st&sort_by=relevance. That's right: two instances of U+FFFD OBJECT REPLACEMENT CHARACTER.

From what I can tell, this isn't due to any badly coded interpretation of mine, but rather this is all the data that the HTTP server gives me. It's not possible to reconstruct the original path.

The previously linked issue is fixed in node.js v11+, so I'll try upgrading to that and see if it's fixed.

@cloudrac3r
Copy link
Owner

Upgraded node.js to v11.2.0 and nothing changed. Not sure what to do now. Guess I'll take a closer look.

@thiswillbeyourgithub
Copy link
Author

I'm not sure but I think I also had the same error while searching for keywords separated by a space like "a b c", I don't know if it was related to it but replacing the space in the adresse url with a plus seemed to solve it (""a+b+c"). It happened a few times for a while but nothing really predictable

@cloudrac3r
Copy link
Owner

I've never had an issue with spaces in searches. If it happens again, I'd appreciate a screenshot and the specific URL that didn't work.

@thiswillbeyourgithub
Copy link
Author

I'll keep that in mind but I don't spend that much time on youtube at this part of the year :/

@cloudrac3r
Copy link
Owner

Fixed the original issue with #252f6e3. Enjoy!

@thiswillbeyourgithub
Copy link
Author

Thanks a lot ! Please feel great today <3

@cloudrac3r
Copy link
Owner

Will do my best ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants