erisod / heattape-smartthings

SmartApp for heat tape control based on weather
Apache License 2.0
1 stars 0 forks source link

API calls denied due to exceeding rate #5

Open dodgeboy opened 5 years ago

dodgeboy commented 5 years ago

It looks like this app has become popular enough that weather underground is denying the API calls due to excessive rate:

conditions data: [response:[error:[description:this key is not valid due to exceeding rate plan, type:invalidkey], features:[:], termsofService:http://www.wunderground.com/weather/api/d/terms.html, version:0.1]]

Is there a way for me to sign up and enter my own key? I thumbed through the code and didn't see where the key was defined. As it stands now, the app does not work at all because it cannot receive weather data.

Edit: It appears this may be related to WUnderground discontinuing their weather API... It looks like the smart apps are supposed to migrate over to getTwcConditions... The WUnderground API was supposed to break on Jan 1 of this year, but this app continued to work until some time in the past week.

Thanks, Dave

dodgeboy commented 4 years ago

Any update on this? It’s getting to be the time of year that I rely on this smart app to control my roof heaters. I tried to modify the code myself but I ran out of talent.

erisod commented 4 years ago

Hi Dave,

I'm sorry I haven't had a chance to look into this. This app is setup to ask Smartthings for the weather data, it doesn't explicitly reach out to wunderground so there is no key involved. I imagine under the covers smartthings uses an api key to construct the query to wunderground and that will need to be sorted out.

Additionally, when I'm trying to login to their developer portal it's not accepting my password nor is the password reset flow working. It's quite f'd up. :(

I'm sorry this isn't working for you (or for me). I will try again to get this working but I'd like to set your expectations to not rely on this software (and perhaps not the platform). FYI I tweeted at smartthings ( https://twitter.com/SFEricM/status/1197656123483930624).

eric

On Sat, Nov 9, 2019 at 1:06 PM dodgeboy notifications@github.com wrote:

Any update on this? It’s getting to be the time of year that I rely on this smart app to control my roof heaters. I tried to modify the code myself but I ran out of talent.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIURPOP4XGFS54PRKK3QS4Q6RA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUPHIY#issuecomment-552137635, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXFHISSJQWWMCGKQ47PHF3QS4Q6RANCNFSM4G2IPY7A .

dodgeboy commented 4 years ago

Thanks for the reply! I’m fairly certain it stopped working because SmartThings changed the way they handle weather calls from smart apps earlier this year. I added an edit to my original message with a comment about that. They want smart apps to use a different method of calling weather because SmartThings themselves teamed up with “The weather company” for their weather data. I tried to rewrite your app to use the new calls (and parse the new results) but I ran out of talent.

On Thu, Nov 21, 2019 at 6:51 PM Eric Mayers notifications@github.com wrote:

Hi Dave,

I'm sorry I haven't had a chance to look into this. This app is setup to ask Smartthings for the weather data, it doesn't explicitly reach out to wunderground so there is no key involved. I imagine under the covers smartthings uses an api key to construct the query to wunderground and that will need to be sorted out.

Additionally, when I'm trying to login to their developer portal it's not accepting my password nor is the password reset flow working. It's quite f'd up. :(

I'm sorry this isn't working for you (or for me). I will try again to get this working but I'd like to set your expectations to not rely on this software (and perhaps not the platform). FYI I tweeted at smartthings ( https://twitter.com/SFEricM/status/1197656123483930624).

eric

On Sat, Nov 9, 2019 at 1:06 PM dodgeboy notifications@github.com wrote:

Any update on this? It’s getting to be the time of year that I rely on this smart app to control my roof heaters. I tried to modify the code myself but I ran out of talent.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub < https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIURPOP4XGFS54PRKK3QS4Q6RA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUPHIY#issuecomment-552137635 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ADXFHISSJQWWMCGKQ47PHF3QS4Q6RANCNFSM4G2IPY7A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=AEMHH36AK7FECCXQW3LBYUTQU4NGLA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4BRZY#issuecomment-557324519, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEMHH36AKGGFAC4HQJQK54LQU4NGLANCNFSM4G2IPY7A .

erisod commented 4 years ago

If I can login to my account I'll see if I can get it working again.

thanks for reporting this issue to me by the way.

On Thu, Nov 21, 2019 at 3:55 PM dodgeboy notifications@github.com wrote:

Thanks for the reply! I’m fairly certain it stopped working because SmartThings changed the way they handle weather calls from smart apps earlier this year. I added an edit to my original message with a comment about that. They want smart apps to use a different method of calling weather because SmartThings themselves teamed up with “The weather company” for their weather data. I tried to rewrite your app to use the new calls (and parse the new results) but I ran out of talent.

On Thu, Nov 21, 2019 at 6:51 PM Eric Mayers notifications@github.com wrote:

Hi Dave,

I'm sorry I haven't had a chance to look into this. This app is setup to ask Smartthings for the weather data, it doesn't explicitly reach out to wunderground so there is no key involved. I imagine under the covers smartthings uses an api key to construct the query to wunderground and that will need to be sorted out.

Additionally, when I'm trying to login to their developer portal it's not accepting my password nor is the password reset flow working. It's quite f'd up. :(

I'm sorry this isn't working for you (or for me). I will try again to get this working but I'd like to set your expectations to not rely on this software (and perhaps not the platform). FYI I tweeted at smartthings ( https://twitter.com/SFEricM/status/1197656123483930624).

eric

On Sat, Nov 9, 2019 at 1:06 PM dodgeboy notifications@github.com wrote:

Any update on this? It’s getting to be the time of year that I rely on this smart app to control my roof heaters. I tried to modify the code myself but I ran out of talent.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <

https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIURPOP4XGFS54PRKK3QS4Q6RA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUPHIY#issuecomment-552137635

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ADXFHISSJQWWMCGKQ47PHF3QS4Q6RANCNFSM4G2IPY7A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=AEMHH36AK7FECCXQW3LBYUTQU4NGLA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4BRZY#issuecomment-557324519 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AEMHH36AKGGFAC4HQJQK54LQU4NGLANCNFSM4G2IPY7A

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIV5775S45RUY26AS4TQU4NXDA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4BZTQ#issuecomment-557325518, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXFHIRJMZLZFSXS4RMGQHTQU4NXDANCNFSM4G2IPY7A .

dodgeboy commented 4 years ago

Thanks for trying to take a look at it! I feel guilty reporting an issue without being able to provide the necessary information to fix it.

On Thu, Nov 21, 2019 at 7:31 PM Eric Mayers notifications@github.com wrote:

If I can login to my account I'll see if I can get it working again.

thanks for reporting this issue to me by the way.

On Thu, Nov 21, 2019 at 3:55 PM dodgeboy notifications@github.com wrote:

Thanks for the reply! I’m fairly certain it stopped working because SmartThings changed the way they handle weather calls from smart apps earlier this year. I added an edit to my original message with a comment about that. They want smart apps to use a different method of calling weather because SmartThings themselves teamed up with “The weather company” for their weather data. I tried to rewrite your app to use the new calls (and parse the new results) but I ran out of talent.

On Thu, Nov 21, 2019 at 6:51 PM Eric Mayers notifications@github.com wrote:

Hi Dave,

I'm sorry I haven't had a chance to look into this. This app is setup to ask Smartthings for the weather data, it doesn't explicitly reach out to wunderground so there is no key involved. I imagine under the covers smartthings uses an api key to construct the query to wunderground and that will need to be sorted out.

Additionally, when I'm trying to login to their developer portal it's not accepting my password nor is the password reset flow working. It's quite f'd up. :(

I'm sorry this isn't working for you (or for me). I will try again to get this working but I'd like to set your expectations to not rely on this software (and perhaps not the platform). FYI I tweeted at smartthings ( https://twitter.com/SFEricM/status/1197656123483930624).

eric

On Sat, Nov 9, 2019 at 1:06 PM dodgeboy notifications@github.com wrote:

Any update on this? It’s getting to be the time of year that I rely on this smart app to control my roof heaters. I tried to modify the code myself but I ran out of talent.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <

https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIURPOP4XGFS54PRKK3QS4Q6RA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUPHIY#issuecomment-552137635

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ADXFHISSJQWWMCGKQ47PHF3QS4Q6RANCNFSM4G2IPY7A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=AEMHH36AK7FECCXQW3LBYUTQU4NGLA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4BRZY#issuecomment-557324519

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AEMHH36AKGGFAC4HQJQK54LQU4NGLANCNFSM4G2IPY7A

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIV5775S45RUY26AS4TQU4NXDA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4BZTQ#issuecomment-557325518 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ADXFHIRJMZLZFSXS4RMGQHTQU4NXDANCNFSM4G2IPY7A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=AEMHH374NAW6ZW72SEAPYPLQU4R5HA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4D5HY#issuecomment-557334175, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEMHH327RKH6QCJG7KHRCXLQU4R5HANCNFSM4G2IPY7A .

erisod commented 4 years ago

Finding is half the solution

On Thu, Nov 21, 2019, 4:42 PM dodgeboy notifications@github.com wrote:

Thanks for trying to take a look at it! I feel guilty reporting an issue without being able to provide the necessary information to fix it.

On Thu, Nov 21, 2019 at 7:31 PM Eric Mayers notifications@github.com wrote:

If I can login to my account I'll see if I can get it working again.

thanks for reporting this issue to me by the way.

On Thu, Nov 21, 2019 at 3:55 PM dodgeboy notifications@github.com wrote:

Thanks for the reply! I’m fairly certain it stopped working because SmartThings changed the way they handle weather calls from smart apps earlier this year. I added an edit to my original message with a comment about that. They want smart apps to use a different method of calling weather because SmartThings themselves teamed up with “The weather company” for their weather data. I tried to rewrite your app to use the new calls (and parse the new results) but I ran out of talent.

On Thu, Nov 21, 2019 at 6:51 PM Eric Mayers notifications@github.com wrote:

Hi Dave,

I'm sorry I haven't had a chance to look into this. This app is setup to ask Smartthings for the weather data, it doesn't explicitly reach out to wunderground so there is no key involved. I imagine under the covers smartthings uses an api key to construct the query to wunderground and that will need to be sorted out.

Additionally, when I'm trying to login to their developer portal it's not accepting my password nor is the password reset flow working. It's quite f'd up. :(

I'm sorry this isn't working for you (or for me). I will try again to get this working but I'd like to set your expectations to not rely on this software (and perhaps not the platform). FYI I tweeted at smartthings ( https://twitter.com/SFEricM/status/1197656123483930624).

eric

On Sat, Nov 9, 2019 at 1:06 PM dodgeboy notifications@github.com wrote:

Any update on this? It’s getting to be the time of year that I rely on this smart app to control my roof heaters. I tried to modify the code myself but I ran out of talent.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <

https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIURPOP4XGFS54PRKK3QS4Q6RA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUPHIY#issuecomment-552137635

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ADXFHISSJQWWMCGKQ47PHF3QS4Q6RANCNFSM4G2IPY7A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=AEMHH36AK7FECCXQW3LBYUTQU4NGLA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4BRZY#issuecomment-557324519

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AEMHH36AKGGFAC4HQJQK54LQU4NGLANCNFSM4G2IPY7A

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIV5775S45RUY26AS4TQU4NXDA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4BZTQ#issuecomment-557325518

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ADXFHIRJMZLZFSXS4RMGQHTQU4NXDANCNFSM4G2IPY7A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=AEMHH374NAW6ZW72SEAPYPLQU4R5HA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4D5HY#issuecomment-557334175 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AEMHH327RKH6QCJG7KHRCXLQU4R5HANCNFSM4G2IPY7A

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/erisod/heattape-smartthings/issues/5?email_source=notifications&email_token=ADXFHIUIK6ZGFKZDXOMCEE3QU4THPA5CNFSM4G2IPY7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE4EQKI#issuecomment-557336617, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXFHISG4SXKH5RWURTWRXTQU4THPANCNFSM4G2IPY7A .