Sets proper 404 status in most cases

This commit is contained in:
jessop 2018-12-13 22:43:03 -05:00
parent c28d45515b
commit d38d8f1061

View file

@ -20,11 +20,11 @@ const createCanonicalLink = require('../../../utils/createCanonicalLink');
const getCanonicalUrlFromShow = show => { const getCanonicalUrlFromShow = show => {
const requestId = show.requestList[show.request.id]; const requestId = show.requestList[show.request.id];
const requestType = show.request.type; const requestType = show.request.type;
if (!requestId || !requestType) { if (!requestId || !requestType) {
return null; return null;
} }
switch (requestType) { switch (requestType) {
case 'ASSET_DETAILS': case 'ASSET_DETAILS':
const asset = show.assetList[requestId.key]; const asset = show.assetList[requestId.key];
@ -57,16 +57,15 @@ module.exports = (req, res) => {
// Workaround, remove when a solution for async httpContext exists // Workaround, remove when a solution for async httpContext exists
const showState = store.getState().show; const showState = store.getState().show;
const assetKeys = Object.keys(showState.assetList); const assetKeys = Object.keys(showState.assetList);
const channelKeys = Object.keys(showState.channelList);
if(assetKeys.length !== 0) {
res.claimId = showState.assetList[assetKeys[0]].claimId;
} else {
const channelKeys = Object.keys(showState.channelList);
if(channelKeys.length !== 0) { if (assetKeys.length !== 0) {
res.claimId = showState.channelList[channelKeys[0]].longId; res.claimId = showState.assetList[assetKeys[0]].claimId;
res.isChannel = true; } else if (channelKeys.length !== 0) {
} res.claimId = showState.channelList[channelKeys[0]].longId;
res.isChannel = true;
} else {
res.status(404);
} }
// render component to a string // render component to a string