Mismatch in url from application.properties and WeatherClient

[micskeiz]

After the update the to openweathermap API, the /weather endpoint is not working. I think the reason is due to weather url.

In application.properties it is defined as:

weather.url = https://api.openweathermap.org/data/2.5/weather

And it is called in WeatherClient.java as:

var url = String.format("%s/data/2.5/weather?q=%s&appid=%s", weatherServiceUrl, CITY, weatherServiceApiKey);

The part /data/2.5/weather will be duplicated. A simple fix is to define it in application.properties as:

weather.url = https://api.openweathermap.org

After fixing it, the existing tests and the /weather endpoint were working for me.

By the way, is /src/main/resources/application-int.properties still used? It still contains a reference to the darksky API.

[hamvocke]

Thanks for spotting! These are indeed two distinct issues and I just fixed both of them:

1. I removed application-int.properties. This was a remnant of an earlier iteration that was no longer referenced in the Readme
2. I fixed the weather.url property in application.properties to not include the URL path