hotel-manage-stack/settings
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

met_office test: full Spot/Forecast x daily/hourly/three-hourly matrix

Browse source 
First guess (Forecast key -> hourly specifically) was tried and confirmed
wrong live. Rather than guess again, the test route now checks every
configured key against every endpoint and reports the full 2x3 matrix, so
the actual working combination is visible on screen instead of assumed.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
jtricerolph 2026-08-25 13:43:24 +00:00
parent 210649d7ad
commit 704c1aa7fa
1 changed files with 32 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

48
src/routes/integrations.js
View file

@ -171,26 +171,40 @@ export async function integrationRoutes(app) {
const { rows: [loc] } = await pool.query(`SELECT config FROM integrations WHERE slug = 'hotel_location'`) const { rows: [loc] } = await pool.query(`SELECT config FROM integrations WHERE slug = 'hotel_location'`)
const lat = loc?.config?.latitude || '51.5072' const lat = loc?.config?.latitude || '51.5072'
const lon = loc?.config?.longitude || '-0.1276' const lon = loc?.config?.longitude || '-0.1276'
// Two separate subscriptions, two separate keys — test each against the // Which of Spot vs Forecast actually covers which timestep isn't
// endpoint weather actually uses it for (Spot -> daily, Forecast -> // documented anywhere we could find, and a first guess (Forecast ->
// hourly). forecast_api_key is optional, only tested if set. // hourly) turned out wrong live — so this tests every configured key
const errors = [] // against every endpoint and reports the full matrix, rather than
const dailyRes = await fetch( // assuming, so the actual working combination is on screen instead of
`https://data.hub.api.metoffice.gov.uk/sitespecific/v0/point/daily?latitude=${lat}&longitude=${lon}`, // guessed at again.
{ headers: { apikey: creds.api_key }, signal: AbortSignal.timeout(8000) } const keys = [['Spot', creds.api_key]]
) if (creds.forecast_api_key) keys.push(['Forecast', creds.forecast_api_key])
if (dailyRes.status === 401 || dailyRes.status === 403) errors.push('Spot key: invalid or not subscribed to the daily endpoint') const endpoints = ['daily', 'hourly', 'three-hourly']
else if (!dailyRes.ok) errors.push(`Spot key: daily endpoint returned ${dailyRes.status}`)
if (creds.forecast_api_key) { const results = []
const hourlyRes = await fetch( for (const [keyName, key] of keys) {
`https://data.hub.api.metoffice.gov.uk/sitespecific/v0/point/hourly?latitude=${lat}&longitude=${lon}`, for (const endpoint of endpoints) {
{ headers: { apikey: creds.forecast_api_key }, signal: AbortSignal.timeout(8000) } let outcome
try {
const res = await fetch(
`https://data.hub.api.metoffice.gov.uk/sitespecific/v0/point/${endpoint}?latitude=${lat}&longitude=${lon}`,
{ headers: { apikey: key }, signal: AbortSignal.timeout(8000) }
) )
if (hourlyRes.status === 401 || hourlyRes.status === 403) errors.push('Forecast key: invalid or not subscribed to the hourly endpoint') if (res.status === 401 || res.status === 403) outcome = 'not subscribed'
else if (!hourlyRes.ok) errors.push(`Forecast key: hourly endpoint returned ${hourlyRes.status}`) else if (!res.ok) outcome = `HTTP ${res.status}`
else outcome = 'OK'
} catch (e) {
outcome = `error: ${e.message}`
} }
if (errors.length) throw new Error(errors.join(' | ')) results.push(`${keyName}/${endpoint}: ${outcome}`)
}
}
// Frontend renders `message` when ok, `error` when not — set both to the
// same diagnostic text so the full matrix shows up either way.
const summary = results.join(' | ')
const ok = results.some(r => r.endsWith('OK'))
return { ok, message: summary, error: summary }
} }
else if (req.params.slug === 'sambapos') { else if (req.params.slug === 'sambapos') {
const errors = [] const errors = []