Page MenuHomePhabricator

Prevent typeahead sources from querying against empty tokens
ClosedPublic

Authored by epriestley on Nov 17 2016, 5:44 PM.
Tags
None
Referenced Files
F14032059: D16888.id40658.diff
Sat, Nov 9, 12:40 PM
F14032058: D16888.id40657.diff
Sat, Nov 9, 12:40 PM
F14032057: D16888.id.diff
Sat, Nov 9, 12:40 PM
F14019785: D16888.diff
Tue, Nov 5, 11:38 PM
F14009562: D16888.id40658.diff
Wed, Oct 30, 7:59 PM
F13985420: D16888.id40657.diff
Sun, Oct 20, 8:25 PM
F13973491: D16888.diff
Oct 18 2024, 1:07 AM
Unknown Object (File)
Sep 12 2024, 5:22 AM
Subscribers

Details

Summary

Certain unusual queries, like [-], could tokenize into a list which included the empty string.

This would then convert into a query for ... LIKE "%" which just joins the entire table.

Instead: tokenize smarter; never return the empty token; add some test cases.

Test Plan

Ran unit tests. Queried for [[blah blah]], saw a reasonable query come out the other end.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Prevent typeahead sources from querying against empty tokens.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a subscriber: 20after4.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 17 2016, 5:44 PM
This revision was automatically updated to reflect the committed changes.