Fix 301 and 302 redirects. (#5999)

packages/volto/news/5999.bugfix

@@ -0,0 +1 @@
+Fix 301 and 302 redirects. @robgietema

packages/volto/src/components/theme/View/View.jsx

@@ -209,6 +209,11 @@ class View extends Component {
     if (this.props.error && this.props.error.code === 301) {
       const redirect = flattenToAppURL(this.props.error.url).split('?')[0];
       return <Redirect to={`${redirect}${this.props.location.search}`} />;
+    } else if (this.props.error && this.props.error.status === 302) {
+      const redirect = flattenToAppURL(
+        this.props.error.response.header.location,
+      ).split('?')[0];
+      return <Redirect to={`${redirect}${this.props.location.search}`} />;
     } else if (this.props.error && !this.props.connectionRefused) {
       let FoundView;
       if (this.props.error.status === undefined) {

packages/volto/src/helpers/Api/Api.js

@@ -54,7 +54,7 @@ class Api {
       ) => {
         let request;
         let promise = new Promise((resolve, reject) => {
-          request = superagent[method](formatUrl(path));
+          request = superagent[method](formatUrl(path)).redirects(0);
 
           if (params) {
             request.query(params);

packages/volto/src/helpers/Api/Api.plone.rest.test.js

@@ -3,12 +3,14 @@ import Api from './Api';
 
 jest.mock('superagent', () => ({
   get: jest.fn((url) => ({
-    url,
-    query: jest.fn(),
-    set: jest.fn(),
-    type: jest.fn(),
-    send: jest.fn(),
-    end: jest.fn(),
+    redirects: jest.fn(() => ({
+      url,
+      query: jest.fn(),
+      set: jest.fn(),
+      type: jest.fn(),
+      send: jest.fn(),
+      end: jest.fn(),
+    })),
   })),
 }));
 

packages/volto/src/helpers/Api/Api.test.js

@@ -9,12 +9,14 @@ import Api from './Api';
 
 jest.mock('superagent', () => ({
   get: jest.fn((url) => ({
-    url,
-    query: jest.fn(),
-    set: jest.fn(),
-    type: jest.fn(),
-    send: jest.fn(),
-    end: jest.fn(),
+    redirects: jest.fn(() => ({
+      url,
+      query: jest.fn(),
+      set: jest.fn(),
+      type: jest.fn(),
+      send: jest.fn(),
+      end: jest.fn(),
+    })),
   })),
 }));