Merge pull request #96 from jagerman/v3-onion-201

jagerman · web-flow · commit 16ff18c52b05 · 2022-03-24T21:06:21.000-03:00
Make sure v3 onion requests work for 201-returning endpoints
diff --git a/sogs/routes/onion_request.py b/sogs/routes/onion_request.py
@@ -87,7 +87,7 @@ def handle_v3_onionreq_plaintext(body):
             user_reauth=True,  # Because onion requests have auth headers on the *inside*
         )
 
-        if response.status_code == http.OK:
+        if 200 <= response.status_code < 300:
             data = response.get_data()
             app.logger.debug(
                 f"Onion sub-request for {endpoint} returned success, {len(data)} bytes"

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ def handle_v3_onionreq_plaintext(body):`
`87`	`87`	`user_reauth=True, # Because onion requests have auth headers on the inside`
`88`	`88`	`)`
`89`	`89`
`90`		`- if response.status_code == http.OK:`
	`90`	`+ if 200 <= response.status_code < 300:`
`91`	`91`	`data = response.get_data()`
`92`	`92`	`app.logger.debug(`
`93`	`93`	`f"Onion sub-request for {endpoint} returned success, {len(data)} bytes"`