ppetru/opensprinkler-weather
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #74 from Derpthemeus/dont-error-on-missing-key

Browse Source 
Silently fallback to default WeatherProvider if WUnderground API key is omitted
This commit is contained in:
Samer Albahra
2019-07-12 16:46:22 -07:00
committed by GitHub
parent 61ecb3a8e0 5787300b87
commit 3038728d4a
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
routes/weather.ts
View File

@@ -219,11 +219,7 @@ export const getWateringData = async function( req: express.Request, res: expres
// Parse the PWS information. // Parse the PWS information.
let pws: PWS | undefined = undefined; let pws: PWS | undefined = undefined;
if ( adjustmentOptions.pws ) { if ( adjustmentOptions.pws && adjustmentOptions.key ) {
if ( !adjustmentOptions.key ) {
res.send("Error: An API key must be provided when using a PWS.");
return;
}
const idMatch = adjustmentOptions.pws.match( /^pws:([a-zA-Z\d]+)$/ ); const idMatch = adjustmentOptions.pws.match( /^pws:([a-zA-Z\d]+)$/ );
const pwsId = idMatch ? idMatch[ 1 ] : undefined; const pwsId = idMatch ? idMatch[ 1 ] : undefined;