From 5758b1687d5cb79beef8b51d90c655657985f6a4 Mon Sep 17 00:00:00 2001
From: kernelpanek <kernelpanek+github@gmail.com>
Date: Wed, 15 Mar 2017 00:25:26 -0600
Subject: [PATCH] Fixes issue when parsing an Nginx configuration file
 containing multiline quoted strings

---
 certbot-nginx/certbot_nginx/nginxparser.py       |  6 +++---
 .../certbot_nginx/tests/nginxparser_test.py      | 15 +++++++++++++++
 .../testdata/etc_nginx/multiline_quotes.conf     | 16 ++++++++++++++++
 3 files changed, 34 insertions(+), 3 deletions(-)
 create mode 100644 certbot-nginx/certbot_nginx/tests/testdata/etc_nginx/multiline_quotes.conf

diff --git a/certbot-nginx/certbot_nginx/nginxparser.py b/certbot-nginx/certbot_nginx/nginxparser.py
index f6437c589..908a6d6cf 100644
--- a/certbot-nginx/certbot_nginx/nginxparser.py
+++ b/certbot-nginx/certbot_nginx/nginxparser.py
@@ -6,7 +6,7 @@ import string
 
 from pyparsing import (
     Literal, White, Word, alphanums, CharsNotIn, Combine, Forward, Group,
-    Optional, OneOrMore, Regex, ZeroOrMore)
+    Optional, OneOrMore, QuotedString, Regex, ZeroOrMore)
 from pyparsing import stringEnd
 from pyparsing import restOfLine
 
@@ -29,8 +29,8 @@ class RawNginxParser(object):
     # any chars in single or double quotes
     # All of these COULD be upgraded to something like
     # https://stackoverflow.com/a/16130746
-    dquoted = Regex(r'(\".*\")')
-    squoted = Regex(r"(\'.*\')")
+    dquoted = QuotedString('"', multiline=True)
+    squoted = QuotedString("'", multiline=True)
     nonspecial = Regex(r"[^\{\};,]")
     varsub = Regex(r"(\$\{\w+\})")
     # nonspecial nibbles one character at a time, but the other objects take
diff --git a/certbot-nginx/certbot_nginx/tests/nginxparser_test.py b/certbot-nginx/certbot_nginx/tests/nginxparser_test.py
index e83b414cf..650090cb2 100644
--- a/certbot-nginx/certbot_nginx/tests/nginxparser_test.py
+++ b/certbot-nginx/certbot_nginx/tests/nginxparser_test.py
@@ -109,6 +109,21 @@ class TestRawNginxParser(unittest.TestCase):
                  ['blah', '"hello;world"'],
                  ['try_files', '$uri @rewrites']]]]]])
 
+    def test_parse_from_file3(self):
+        with open(util.get_data_filename('multiline_quotes.conf')) as handle:
+            parsed = util.filter_comments(load(handle))
+        self.assertEqual(
+            parsed,
+            [[['http'],
+                [[['server'],
+                    [['listen', '*:443'],
+                    [['location', '/'],
+                        [['body_filter_by_lua',
+                          'ngx.ctx.buffered = (ngx.ctx.buffered or "") .. string.sub(ngx.arg[1], 1, 1000)\n'
+                          '                            if ngx.arg[2] then\n'
+                          '                              ngx.var.resp_body = ngx.ctx.buffered\n'
+                          '                            end']]]]]]]])
+
     def test_abort_on_parse_failure(self):
         with open(util.get_data_filename('broken.conf')) as handle:
             self.assertRaises(ParseException, load, handle)
diff --git a/certbot-nginx/certbot_nginx/tests/testdata/etc_nginx/multiline_quotes.conf b/certbot-nginx/certbot_nginx/tests/testdata/etc_nginx/multiline_quotes.conf
new file mode 100644
index 000000000..74cd84bcd
--- /dev/null
+++ b/certbot-nginx/certbot_nginx/tests/testdata/etc_nginx/multiline_quotes.conf
@@ -0,0 +1,16 @@
+# Test nginx configuration file with multiline quoted strings.
+# Good example of usage for multilined quoted values is when
+# using Openresty's Lua directives and you wish to keep the
+# inline Lua code readable.
+http {
+  server {
+      listen *:443; # because there should be no other port open.
+
+      location / {
+        body_filter_by_lua 'ngx.ctx.buffered = (ngx.ctx.buffered or "") .. string.sub(ngx.arg[1], 1, 1000)
+                            if ngx.arg[2] then
+                              ngx.var.resp_body = ngx.ctx.buffered
+                            end';
+      }
+  }
+}