Slight improvments to error messaging for sitemap and quality report

This commit is contained in:
Alicia Sykes 2024-03-14 07:59:03 +00:00
parent 41cd379805
commit ee74c5866a
3 changed files with 10 additions and 9 deletions

View File

@ -5,14 +5,17 @@ const handler = async (url, event, context) => {
const apiKey = process.env.GOOGLE_CLOUD_API_KEY; const apiKey = process.env.GOOGLE_CLOUD_API_KEY;
if (!apiKey) { if (!apiKey) {
throw new Error('API key (GOOGLE_CLOUD_API_KEY) not set'); throw new Error(
'Missing Google API. You need to set the `GOOGLE_CLOUD_API_KEY` environment variable'
);
} }
const endpoint = `https://www.googleapis.com/pagespeedonline/v5/runPagespeed?url=${encodeURIComponent(url)}&category=PERFORMANCE&category=ACCESSIBILITY&category=BEST_PRACTICES&category=SEO&category=PWA&strategy=mobile&key=${apiKey}`; const endpoint = `https://www.googleapis.com/pagespeedonline/v5/runPagespeed?`
+ `url=${encodeURIComponent(url)}&category=PERFORMANCE&category=ACCESSIBILITY`
+ `&category=BEST_PRACTICES&category=SEO&category=PWA&strategy=mobile`
+ `&key=${apiKey}`;
const response = await axios.get(endpoint); return (await axios.get(endpoint)).data;
return response.data;
}; };
module.exports = middleware(handler); module.exports = middleware(handler);

View File

@ -39,10 +39,8 @@ const handler = async (url) => {
return sitemap; return sitemap;
} catch (error) { } catch (error) {
// If error occurs
console.log(error.message);
if (error.code === 'ECONNABORTED') { if (error.code === 'ECONNABORTED') {
return { error: 'Request timed out' }; return { error: 'Request timed out after 5000ms' };
} else { } else {
return { error: error.message }; return { error: error.message };
} }

View File

@ -288,7 +288,7 @@ const Results = (): JSX.Element => {
addressInfo: { address, addressType, expectedAddressTypes: urlTypeOnly }, addressInfo: { address, addressType, expectedAddressTypes: urlTypeOnly },
fetchRequest: () => fetch(`${api}/quality?url=${address}`) fetchRequest: () => fetch(`${api}/quality?url=${address}`)
.then(res => parseJson(res)) .then(res => parseJson(res))
.then(res => res?.lighthouseResult || { error: 'No Data'}), .then(res => res?.lighthouseResult || { error: res.error || 'No Data' }),
}); });
// Get the technologies used to build site, using Wappalyzer // Get the technologies used to build site, using Wappalyzer