Server-side redirects for plone.app.redirector aliases (v17) (#6154)

news/4854.bugfix

@@ -0,0 +1 @@
+Return a redirect response from Volto server-side rendering if the API request was redirected. @JeffersonBledsoe @mamico

src/components/theme/View/View.jsx

@@ -205,8 +205,10 @@ class View extends Component {
  */
  render() {
  const { views } = config;
- if (this.props.error && this.props.error.code === 301) {
- const redirect = flattenToAppURL(this.props.error.url).split('?')[0];
+ if ([301, 302].includes(this.props.error?.code)) {
+ const redirect = flattenToAppURL(this.props.error.url)
+ .split('?')[0]
+ .replace('/++api++', '');
  return <Redirect to={`${redirect}${this.props.location.search}`} />;
  } else if (this.props.error && !this.props.connectionRefused) {
  let FoundView;

src/components/theme/View/View.test.jsx

@@ -22,6 +22,7 @@ beforeAll(() => {
  });
  config.settings.publicURL = 'https://plone.org';
 });
+global.__SERVER__ = true; // eslint-disable-line no-underscore-dangle
 
 const mockStore = configureStore();
 

src/helpers/Api/Api.js

@@ -80,6 +80,10 @@ class Api {
 
  Object.keys(headers).forEach((key) => request.set(key, headers[key]));
 
+ if (__SERVER__ && checkUrl && ['get', 'head'].includes(method)) {
+ request.redirects(0);
+ }
+
  if (data) {
  request.send(data);
  }
@@ -104,6 +108,14 @@ class Api {
  url: request.xhr.responseURL,
  });
  }
+
+ if ([301, 302].includes(err?.status)) {
+ return reject({
+ code: err.status,
+ url: err.response.headers.location,
+ });
+ }
+
  return err ? reject(err) : resolve(response.body || response.text);
  });
  });

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

@@ -18,6 +18,7 @@ beforeAll(() => {
 
 const api = new Api();
 const { settings } = config;
+global.__SERVER__ = true; // eslint-disable-line no-underscore-dangle
 
 test('get request', () => {});
 

src/helpers/Api/Api.test.js

@@ -24,6 +24,7 @@ beforeAll(() => {
 
 const api = new Api();
 const { settings } = config;
+global.__SERVER__ = true; // eslint-disable-line no-underscore-dangle
 
 test('get request', () => {});