search

grahamwetzler / smart-meter-texas

Package to connect to and retrieve data from the unoffical Smart Meter Texas API
MIT License
14 stars 14 forks source link

Update const.py #135

Closed meyerrj closed 1 year ago

meyerrj commented 1 year ago

Changes the base hostname to just smartmetertexas.com

The base domain is now part of the SSL certificate's SAN: https://github.com/home-assistant/core/issues/57582#issuecomment-1287929568

This change should help resolve the issue here: https://github.com/home-assistant/core/issues/57582

grahamwetzler commented 1 year ago

Is this needed? I was able to connect successfully today.

meyerrj commented 1 year ago

I'll try testing against the www. subdomain again, but last I had attempted about a month or so ago I appeared to be blocked by Akamai. I've only been successful with the base domain as the target URL.

grahamwetzler commented 1 year ago

@meyerrj Does this work for you after the recent change to the server? This branch is still failing for me

meyerrj commented 1 year ago

Sorry, I haven't had a chance to test.

It appears to be broken all around though at the moment, seems like they changed something on the servers again...

meyerrj commented 1 year ago

The base domain now appears to terminate any HTTPS connections. I'm getting a PR_CONNECT_RESET_ERROR with Firefox. Not even getting response headers. I can also confirm that the www subdomain is still blocking me as well (Akamai).

https://github.com/home-assistant/core/issues/57582 has a lot of others reporting other broken integrations as well over the past few days, even those that used Node-RED. 😣

grahamwetzler commented 1 year ago

That's unfortunate 😞. Thanks for investigating.

utdrmac commented 1 year ago

Since this hasn't been merged, I'm going to toss in my confirmation that this PR fixes the problem. I copied the examples from this repo, supplied my un/pw, and neither example worked. I manually merged this PR to my pip install and the examples now work.

grahamwetzler commented 1 year ago

Confirmed it does work. I'll go ahead and merge.

meyerrj commented 1 year ago

Glad to see that this patch still works, and thanks for merging it in! :)