Authentication broken for SAJ API #10
Comments
|
I am having same issue for a week now. |
|
I no longer have a SAJ inverter, but it looks like SAJ eSolar has changed login. The login link the integration use responds with 302. It looks like they have moved the login and in the portal they seem to add CAPTCH for some market. Likely this integration will no longer work due to this. The other SAJ integration seems to have same/similar issue: Please try it instead. Since I don't have a SAJ inverter it is unlikely I will spend time on this... Feel free to nominate someone to continue this work... |
|
That was my assumption as I noticed login page was different. I tried other
saj integration (which I had before yours) .
As you replied no go shame .
Thanks for your reply .
[image: Mailtrack]
<https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&>
Sender
notified by
Mailtrack
<https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&>
01/03/24,
07:10:38 PM
…On Tue, 2 Jan 2024 at 17:56, faanskit ***@***.***> wrote:
I no longer have a SAJ inverter, but it looks like SAJ eSolar has changed
login. The login link the integration use responds with 302.
image.png (view on web)
<https://github.com/faanskit/ha-esolar/assets/10439868/1509ad7f-46fa-4782-81d2-784696f82969>
It looks like they have moved the login and in the portal they seem to add
CAPTCH for some market.
Likely this integration will no longer work due to this.
The other SAJ integration seems to have same/similar issue:
djansen1987/SAJeSolar#72
<djansen1987/SAJeSolar#72>
Please try it instead.
Since I don't have a SAJ inverter it is unlikely I will spend time on
this...
Feel free to nominate someone to continue this work...
—
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASEBYSD4QTDXGQETLODX3E3YMO4S3AVCNFSM6AAAAABBIXY6K6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZTG4YDMOBZGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
Regards from
David Watson
0416505953
|
|
How do I find someone who could fix this issue? |
|
There are some forks made, check with the owners of the forks if they have a solution. I noticed that they also started with CAPTCHA for some regions now, so likely soon it will completely break. You can always ask Saj to publish API's...the more who asks, maybe one day they will support it. SolaX did it after community push. |
|
As SAJ is (partial) a chinese company, I think we can forget about a public API. My integration is currently still working, fingers crossed that it stays like this. |
|
@robshot do you have a link for your integration? |
It's in the first sentence of the README to this integration :-D |
|
I mean the integration in my HA, I'm using this intergration in my HA as I helped faanskit with debugging some things. |
|
ha ha, yeah, bit of a misunderstanding here. I thought @robshot was saying he had a separate HA integration (fork or whatever) that was still able to connect to the SAJ API. |
|
This is an old issue, but the API authentication is not broken. The problem is that SAJ provides two nodes:
This application defaults to the EU node, while the international node has different URLs. I'm working on a fix and will submit a PR, but the easy hack is to go to your custom_components/saj_esolar_air folder and edit esolar.py lines 11 and 12 and replace it with: BASE_URL = "https://intopapp.saj-electric.com/sajAppApi/api" Also remove the pycache folder under the same folder. Restart homeassistant and reconfigure it again. It worked just fine for me.. |
|
@lucianor if you want to take over this repository, let me know and I´ll transfer it to you or make you an admin. |
|
I just added a PR to fix the region switch problem, so this bug can be closed. |
|
@djansen1987 is another good candidate to maintain this. It looks like he did not fork so now I'm trying to compare his repository with this, but his code structure is different. His field handling however is better. So I think we need to combine the two somehow. If @djansen1987 is game, so am I. |
|
Hi for me goes the same, I only created the integration because of a family member. It currently does everything he needs. If some one wants to take over be my guest. Only requirement I have is it should be backwards compatible so the current 106 users won't lose their history data (longterm data which is held to the entity id) |
|
Fair point.. Will submit another PR to address that and defaults to EU region, since that is the default for this till now. If the sensor data changes from your fork and this, then there is a higher challenge, and maybe we will need to have two sensors with same data. |
|
Something like that yes. Both integrations have nice features, hard part is to bring them together. Even nicer would be if the original integration in home assistant (official) could be updated so all users can benefit without having to install hacs, but this will cost some tike and effort. Also some migration documentation. |
|
There is where I'm heading next... |
Contacted you via e-mail and made you a collaborator. |
|
Please see if pre-release V0.0.10 solves this. PR from @lucianor merged |
|
@lucianor if you see any options in to merging this project and mine, you are welcome to have a chat about it |
Hi, trying to add my B2-HV system to HA but getting invalid credentials when trying to add the intergration. Is there a problem with the integration with the new API or is it me?
The text was updated successfully, but these errors were encountered: