From ec0240395f1ea22d47b5860cc449894a23a46f5d Mon Sep 17 00:00:00 2001 From: Simon Detheridge Date: Mon, 28 Sep 2020 11:50:23 +0100 Subject: [PATCH] Merge pull request #3219 from overleaf/spd-jel-v1-logging Add URL to error log from V1SubscriptionManager GitOrigin-RevId: de55b3c017aa060a75d81e538be0dec3131b3baf --- .../src/Features/Subscription/V1SubscriptionManager.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/services/web/app/src/Features/Subscription/V1SubscriptionManager.js b/services/web/app/src/Features/Subscription/V1SubscriptionManager.js index 42e484c2fe..20c44171ba 100644 --- a/services/web/app/src/Features/Subscription/V1SubscriptionManager.js +++ b/services/web/app/src/Features/Subscription/V1SubscriptionManager.js @@ -153,10 +153,11 @@ module.exports = V1SubscriptionManager = { if (v1Id == null) { return callback(null, null, null) } + const url = options.url(v1Id) return request( { baseUrl: settings.apis.v1.url, - url: options.url(v1Id), + url, method: options.method, auth: { user: settings.apis.v1.user, @@ -169,7 +170,10 @@ module.exports = V1SubscriptionManager = { function(error, response, body) { if (error != null) { return callback( - new V1ConnectionError('no v1 connection').withCause(error) + new V1ConnectionError({ + message: 'no v1 connection', + info: { url } + }).withCause(error) ) } if (response && response.statusCode >= 500) {