Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

It seems that the used URL no longer seems to be available from the command line #175

Closed
Henkeh opened this issue Nov 29, 2022 · 8 comments · Fixed by #177
Closed

It seems that the used URL no longer seems to be available from the command line #175

Henkeh opened this issue Nov 29, 2022 · 8 comments · Fixed by #177
Assignees

Comments

@Henkeh
Copy link

Henkeh commented Nov 29, 2022

When I try to readout some values using your library, I keep getting the following response:

ERROR:root:RequestException: 403 Client Error: Forbidden for url: https://semsportal.com/api/v1/PowerStation/GetMonitorDetailByPowerstationId
ERROR:root:RequestException: 403 Client Error: Forbidden for url: https://semsportal.com/api/v1/PowerStation/GetMonitorDetailByPowerstationId
ERROR:root:Failed to call GoodWe API url='https://semsportal.com/api/v1/PowerStation/GetMonitorDetailByPowerstationId'

I have been using it successfully for quite a while now, but recently it stopped working. Do you have any idea how to solve this? Or am I doing something wrong?

@klara31
Copy link

klara31 commented Nov 29, 2022

Same here...

@yaleman
Copy link
Owner

yaleman commented Nov 29, 2022

How recently did it stop working? Today? A week ago? Either way, I'm really not sure as I don't run the API. Maybe someone can work out what's changed?

@klara31
Copy link

klara31 commented Nov 29, 2022

Last data I received was yesterday around 13.00 (CET)

@klara31
Copy link

klara31 commented Nov 29, 2022

When globally changing the API endpoint URL in pygoodwe/__init__.py from v1 to v2, it works again

@Henkeh
Copy link
Author

Henkeh commented Nov 29, 2022

That seems to work for me too.

@yaleman yaleman self-assigned this Nov 29, 2022
@yaleman
Copy link
Owner

yaleman commented Nov 29, 2022

That's not working for me, can you please submit a PR with your changes?

Gregor-at-GitHub added a commit to Gregor-at-GitHub/pygoodwe that referenced this issue Nov 30, 2022
API Version jump from V1 to V2 should fix the issue yaleman#175
@tywjohn
Copy link
Contributor

tywjohn commented Dec 18, 2022

I can confirm Gregor-at-GitHub's change works for me

@yaleman
Copy link
Owner

yaleman commented Dec 18, 2022

Fixes have been merged, new release on PyPI as 0.1.2 thanks for finding the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants