From: Michael Orlitzky <michael@orlitzky.com>
Date: Sun, 1 Jan 2017 22:42:59 +0000 (-0500)
Subject: Remove a querystring parameter to fix v12.2 backups.
X-Git-Tag: 0.0.6~1
X-Git-Url: http://gitweb.michael.orlitzky.com/?p=untangle-https-backup.git;a=commitdiff_plain;h=b5f616878bac3673de90e9c47bf8dedbae790dfd;ds=sidebyside

Remove a querystring parameter to fix v12.2 backups.

In the v10 backup routine, we were passing "type=backup" as part of
the querystring in addition to POSTing it. This was causing a problem
with one v12.2 host, and that's why I'm just going to blame v12.2 for
the problem. It looks like the querystring parameter is not needed for
any of the other versions/hosts that I've tested against.
---

diff --git a/src/untangle/untangle.py b/src/untangle/untangle.py
index 31ab60e..c66cd67 100644
--- a/src/untangle/untangle.py
+++ b/src/untangle/untangle.py
@@ -107,7 +107,7 @@ class Untangle:
 
         Returns the binary HTTPS response (i.e. the file).
         """
-        url = self.base_url + '/webui/download?type=backup'
+        url = self.base_url + '/webui/download'
         post_vars = {'type': 'backup'}
         post_data = urllib.parse.urlencode(post_vars).encode('ascii')
         with self.opener.open(url, post_data) as response: