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

[IQDB] URL broken #11

Closed
Xanawatt opened this issue Dec 19, 2020 · 8 comments
Closed

[IQDB] URL broken #11

Xanawatt opened this issue Dec 19, 2020 · 8 comments

Comments

@Xanawatt
Copy link

the link appears like this when I do a search:
image

it opens like that in the browser as well, leading to a broken page

@Decimation
Copy link
Owner

This is normal behavior. The result is supposed to be the most accurate result link, not necessarily the search link (i.e. https://iqdb.org/?url=.... I have been considering adding the ability to open the search link, which is what I'm guessing you want?

leading to a broken page

What do you mean by this?

@Xanawatt
Copy link
Author

no, it's because of the http:https://
This page doesn't exist because when opened in a browser, the page gives http:https//yande.re/post/show/589623
Somewhere, it looks like you're appending http or something.

@Xanawatt
Copy link
Author

btw, in regards to your assumption. I like what you did with the saucenao results with holding alt to get the other results. (It maybe could be applied to other search engines)

@Decimation
Copy link
Owner

Oh, woops, I didn't see that.

You're correct, I append "http:" to IQDB's results. It turns out that some results from IQDB already have "http:" prefixes, which is why that bug occurred. I'll have a fix for this soon.

@Decimation
Copy link
Owner

@Xanawatt Ok, try v2.1.

@Xanawatt
Copy link
Author

yup, worked perfectly! Thanks!

@Decimation
Copy link
Owner

Great! No problem.

Also:

btw, in regards to your assumption. I like what you did with the saucenao results with holding alt to get the other results. (It maybe could be applied to other search engines)

It's actually used for all engines, so long as there are other results (that can be parsed).

@Xanawatt
Copy link
Author

Ah, I see now. Nice!

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

No branches or pull requests

2 participants