yuuza/outline
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Implement 404 for documents

Browse Source
This commit is contained in:
Jori Lallo
2016-08-03 15:36:50 +03:00
parent 80edf4a693
commit 5b3ed8b1f9
2 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
frontend/utils/ApiClient.js
View File

@@ -1,4 +1,5 @@
import _ from 'lodash'; import _ from 'lodash';
import { browserHistory } from 'react-router';
import stores from 'stores'; import stores from 'stores';
import constants from '../constants'; import constants from '../constants';
@@ -57,6 +58,11 @@ class ApiClient {
return response; return response;
} }
// Handle 404
if (response.status === 404) {
return browserHistory.push('/404');
}
// Handle 401, log out user // Handle 401, log out user
if (response.status === 401) { if (response.status === 401) {
stores.user.logout(); stores.user.logout();

4
server/api/documents.js
View File

@@ -22,6 +22,8 @@ router.post('documents.info', auth({ require: false }), async (ctx) => {
}, },
}); });
if (!document) throw httpErrors.NotFound();
// Don't expose private documents outside the team // Don't expose private documents outside the team
if (document.private) { if (document.private) {
if (!ctx.state.user) throw httpErrors.NotFound(); if (!ctx.state.user) throw httpErrors.NotFound();
@@ -39,8 +41,6 @@ router.post('documents.info', auth({ require: false }), async (ctx) => {
data: await presentDocument(document), data: await presentDocument(document),
}; };
} }
if (!document) throw httpErrors.NotFound();
}); });
router.post('documents.search', auth(), async (ctx) => { router.post('documents.search', auth(), async (ctx) => {