Don't unconditionally search for http://example.com/ #2334

Merged
merged 1 commit into from Jun 26, 2015

Projects

None yet

4 participants

@nickstenning
Member

When we fail to find a document for the passed URI, we should probably
fall back to a naive match query for the passed URI, rather than a naive
match query for the literal string "http://example.com/".

...!

Fixes #2326.

@nickstenning nickstenning Don't unconditionally search for http://example.com/
When we fail to find a document for the passed URI, we should probably
fall back to a naive match query for the passed URI, rather than a naive
match query for the literal string "http://example.com/".
bde0c04
@landscape-bot

Code Health
Code quality remained the same when pulling bde0c04 on hypothesis:2326-fix-search-with-no-document into c50b946 on hypothesis:master.

@judell judell merged commit ccbc673 into master Jun 26, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@nickstenning nickstenning deleted the 2326-fix-search-with-no-document branch Jun 26, 2015
@tilgovi
Contributor
tilgovi commented Jun 26, 2015

Good find :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment