Handle broken links better

This commit is contained in:
Jonathan Cremin 2019-03-11 23:26:54 +00:00
parent b5f99b217a
commit 3abe8f1a79
4 changed files with 93 additions and 70 deletions

View file

@ -91,7 +91,9 @@ export async function lookupId(id, type) {
}, },
}); });
} }
return Promise.reject(new Error()); const error = new Error('Not Found');
error.status = 404;
return Promise.reject(error);
} }
export async function search(data, original = {}) { export async function search(data, original = {}) {

View file

@ -23,6 +23,7 @@ let ready = pm.initAsync(creds).catch(function(err) {
export async function lookupId(id, type) { export async function lookupId(id, type) {
await ready; await ready;
try {
if (type === 'album') { if (type === 'album') {
const album = await pm.getAlbumAsync(id, false); const album = await pm.getAlbumAsync(id, false);
return { return {
@ -61,7 +62,14 @@ export async function lookupId(id, type) {
}, },
}; };
} }
return { service: 'google' }; } catch(e) {
const error = new Error('Not Found');
error.status = 404;
return Promise.reject(error);
}
const error = new Error('Not Found');
error.status = 404;
return Promise.reject(error);
} }
function exactMatch(needle, haystack, type) { function exactMatch(needle, haystack, type) {

View file

@ -39,13 +39,13 @@ export async function lookupId(possibleId, type, countrycode) {
if (cc) { if (cc) {
path = `/${cc}${path}`; path = `/${cc}${path}`;
} }
try {
const response = await request.get(apiRoot + path); const response = await request.get(apiRoot + path);
let result = JSON.parse(response.text); let result = JSON.parse(response.text);
if (!result.results || result.resultCount === 0 || !result.results[0].collectionId) { if (!result.results || result.resultCount === 0 || !result.results[0].collectionId) {
const error = new Error('Not Found'); throw new Error();
error.status = 404;
throw error;
} else { } else {
result = result.results[0]; result = result.results[0];
@ -73,6 +73,11 @@ export async function lookupId(possibleId, type, countrycode) {
return item; return item;
} }
} catch(e) {
const error = new Error('Not Found');
error.status = 404;
return Promise.reject(error);
}
} }
export async function search(data) { export async function search(data) {

View file

@ -44,6 +44,12 @@ export async function lookupId(id, type) {
data = data.body[`${type}s`][0]; data = data.body[`${type}s`][0];
if(!data) {
const error = new Error('Not Found');
error.status = 404;
return Promise.reject(error);
}
const artist = data.artists[0]; const artist = data.artists[0];
if (type === 'album') { if (type === 'album') {
@ -82,7 +88,9 @@ export async function lookupId(id, type) {
}, },
}; };
} }
return { service: 'spotify' }; const error = new Error('Not Found');
error.status = 404;
return Promise.reject(error);
} }
export async function search(data, original = {}) { export async function search(data, original = {}) {