Merge pull request #39 from codefresh-io/CR-15973-debug

dmelnyk-octo · web-flow · commit a235ff18a966 · 2022-12-13T12:31:58.000+02:00
catch 302 redirects
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "nodegistry",
-  "version": "1.2.3",
+  "version": "1.2.4",
   "description": "Docker Registry Remote API module.",
   "main": "index.js",
   "repository": "git@github.com:codefresh-io/nodegistry.git",
diff --git a/src/lib/Modem.js b/src/lib/Modem.js
@@ -110,7 +110,7 @@ exports.RegistryModem = class {
                 });
             }))
             .then(([response, body]) => {
-                if (response.statusCode === 307) {
+                if ([302, 307].includes(response.statusCode)) {
                     return this.handleRedirect(response.headers.location);
                 }
                 return this._promise.resolve([response, body]);

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "nodegistry",`
`3`		`- "version": "1.2.3",`
	`3`	`+ "version": "1.2.4",`
`4`	`4`	`"description": "Docker Registry Remote API module.",`
`5`	`5`	`"main": "index.js",`
`6`	`6`	`"repository": "git@github.com:codefresh-io/nodegistry.git",`
Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ exports.RegistryModem = class {`
`110`	`110`	`});`
`111`	`111`	`}))`
`112`	`112`	`.then(([response, body]) => {`
`113`		`- if (response.statusCode === 307) {`
	`113`	`+ if ([302, 307].includes(response.statusCode)) {`
`114`	`114`	`return this.handleRedirect(response.headers.location);`
`115`	`115`	`}`
`116`	`116`	`return this._promise.resolve([response, body]);`