Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

[weather] Fix OpenWeather API key failure by removing FORECAST #5838

Closed
wants to merge 3 commits into from

Commits on Mar 21, 2019

  1. [Weather] openhab#5837 Fix OpenWeather API key failure

    Why:
    * OpenWeather discontinued the FreePlan for FORECAST which now generates an API-key error as forecast is implicitly part of binding.
    This Fix will remove the FORECAST call and option by simply returning nulll for the FORECASY url.
    
    This change addresses the need by:
    * return null in the FORECAST call
    
    Signed-off-by: PtrO <[email protected]>
    ptrooms committed Mar 21, 2019
    Configuration menu
    Copy the full SHA
    a2756a3 View commit details
    Browse the repository at this point in the history

Commits on Mar 23, 2019

  1. Solve API keyfailure when using OpenWweatherMap, Forecast (requires a…

    … paid key) is only queried if using apikey2 in weather.config.
    
    Note: apikey2 is also forcily made "absent" when specified als apikey2=null (exact string).
    This as solely removing the apikey2 from the weather.cfg file will not remove the value from cached settings.
    
    To summarize:
    if one has obtained a Forecast validated key from OpenWeather, specify this key in config apikey2=<key-value>
    Incase one has only the Freeplan, do not specify apikey2 or code it als apikey2=null, to skip forecasts.
    
    Signed-off-by: PtrO <[email protected]>
    ptrooms committed Mar 23, 2019
    Configuration menu
    Copy the full SHA
    12f09f1 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    1340931 View commit details
    Browse the repository at this point in the history