Page MenuHomePhabricator

Search backend error during full_text search on forward slash
Closed, ResolvedPublic


Multiple PHP warnings appeared in error log after enabling CirrusSearch. This is the most common:

21 PHP Warning:  Search backend error during full_text search for '1851d4395efd9e677744fbb61b05ee7e2b9e8b45-With this form you can create/en' after 10.  Parse error on '1851d4395efd9e677744fbb61b05ee7e2b9e8b45\-With this form you can create/en': Lexical error at line 1, column 75.  Encountered: <EOF> after : "/en"]; nested: TokenMgrError[Lexical error at line 1, column 75.  Encountered: <EOF> after : "/en"]; }{[2AB6CtULQdaQz1HlOe7BMg][translatewiki_net-bw__content_first][3]: SearchParseException[[translatewiki_net-bw__content_first][3]: from[-1],size[-1]: Parse Failure [Failed to parse source [{"_source":["id","title","namespace","redirect.*","timestamp","text_bytes"],"fields":"text.word_count","query":{"filtered":{"query":{"bool":{"minimum_number_should_match":1,"should":[{"query_string":{"query":"1851d4395efd9e677744fbb61b05ee7e2b9e8b45\\-With this form you can create\/en","fields":["title.plain^20","redirect.title.plain^15","category.plain^8","heading.plain^5","opening_text.plain^4","text.plain^1","auxiliary_text.plain^0.8 in /www/ on line 303

Frequency of this warning is 201 per 367 (55%).

Further info:

The page in question: has one revision which has been suppressed.

Version: master
Severity: normal



Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

bzimport raised the priority of this task from to High.
bzimport set Reference to bz67601.

Looks like a parse error on the stupid query string. I honestly thought we'd worked through them all in the time since we've been testing cirrus on the cluster. It looks like its trying to kick off a regex query starting at "/en" and failing because its missing the trailing "/". Cute.

I can't get it to fail on test2 and I'm in the middle of rebuilding my vagrant box. Once its done I'll give this a shot locally.

Scratch that. Reproduced locally.

Change 144477 had a related patch set uploaded by Manybubbles:
Escape forward slashes a bit harder

Approved by Nikerabbit.

Change 144477 merged by jenkins-bot:
Escape forward slashes a bit harder