From 551b8fe0267e6ff6e535524f16e789a3eb527e06 Mon Sep 17 00:00:00 2001 From: Alex Osborne Date: Thu, 13 May 2021 10:33:58 +0900 Subject: [PATCH] xmlquery: remove space after the "limit:" query field name (#640) OutbackCDX can't handle a space here as it decodes fields by splitting on space. --- pywb/warcserver/index/indexsource.py | 2 +- pywb/warcserver/index/test/test_xmlquery_indexsource.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pywb/warcserver/index/indexsource.py b/pywb/warcserver/index/indexsource.py index e5e3f7bb..ced10f15 100644 --- a/pywb/warcserver/index/indexsource.py +++ b/pywb/warcserver/index/indexsource.py @@ -248,7 +248,7 @@ class XmlQueryIndexSource(BaseIndexSource): try: limit = params.get('limit') if limit: - query = 'limit: {0} '.format(limit) + query + query = 'limit:{0} '.format(limit) + query # OpenSearch API requires double-escaping # TODO: add option to not double escape if needed diff --git a/pywb/warcserver/index/test/test_xmlquery_indexsource.py b/pywb/warcserver/index/test/test_xmlquery_indexsource.py index 63f832ce..3986d11e 100644 --- a/pywb/warcserver/index/test/test_xmlquery_indexsource.py +++ b/pywb/warcserver/index/test/test_xmlquery_indexsource.py @@ -78,7 +78,7 @@ com,example)/ 20180112200243 example.warc.gz com,example)/ 20180216200300 example.warc.gz""" assert(key_ts_res(reslist) == expected) assert(errs == {}) - assert query_url == 'http://localhost:8080/path?q=limit%3A+100+type%3Aurlquery+url%3Ahttp%253A%252F%252Fexample.com%252F' + assert query_url == 'http://localhost:8080/path?q=limit%3A100+type%3Aurlquery+url%3Ahttp%253A%252F%252Fexample.com%252F' assert reslist[0]['length'] == '123' assert 'length' not in reslist[1]