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

Stop Working #31

Open
milandzuris opened this issue May 7, 2023 · 38 comments
Open

Stop Working #31

milandzuris opened this issue May 7, 2023 · 38 comments
Assignees
Labels
bug Something isn't working

Comments

@milandzuris
Copy link

Integration Stop Working, have problem with login to google, i think is good make new integration and setup from UI.

@BebeMischa
Copy link

BebeMischa commented May 7, 2023

Logger: homeassistant.components.sensor
Source: custom_components/google_keep/sensor.py:39
Integration: Sensor (documentation, issues)
First occurred: 6 mei 2023 om 03:33:03 (1 occurrences)
Last logged: 6 mei 2023 om 03:33:03

Error while setting up google_keep platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 304, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/google_keep/sensor.py", line 39, in setup_platform
    login_success = keep.login(username, password)
  File "/usr/local/lib/python3.10/site-packages/gkeepapi/__init__.py", line 693, in login
    ret = auth.login(username, password, get_mac())
  File "/usr/local/lib/python3.10/site-packages/gkeepapi/__init__.py", line 59, in login
    raise exception.LoginException(
gkeepapi.exception.LoginException: ('BadAuthentication', None)

@PiotrMachowski , would you please have a look in to this? Thank you very much in advance. ;-)

@milandzuris
Copy link
Author

Logger: homeassistant.components.sensor
Source: custom_components/google_keep/sensor.py:39
Integration: Sensor (documentation, issues)
First occurred: 6 mei 2023 om 03:33:03 (1 occurrences)
Last logged: 6 mei 2023 om 03:33:03

Error while setting up google_keep platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 304, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/google_keep/sensor.py", line 39, in setup_platform
    login_success = keep.login(username, password)
  File "/usr/local/lib/python3.10/site-packages/gkeepapi/__init__.py", line 693, in login
    ret = auth.login(username, password, get_mac())
  File "/usr/local/lib/python3.10/site-packages/gkeepapi/__init__.py", line 59, in login
    raise exception.LoginException(
gkeepapi.exception.LoginException: ('BadAuthentication', None)

@PiotrMachowski , would you please have a look in to this? Thank you very much in advance. ;-)

same

@superharaldsson
Copy link

Same here!

@yjamal01
Copy link

yjamal01 commented May 8, 2023

same issue here since 2023.5

@AlexKusnezov
Copy link

same here

@PiotrMachowski PiotrMachowski self-assigned this May 8, 2023
@milandzuris
Copy link
Author

this problem i have last month i think from core 2023.4

@milandzuris
Copy link
Author

you start working on this problem or is time to delete? (question) thx

@PiotrMachowski PiotrMachowski added the bug Something isn't working label May 8, 2023
@PiotrMachowski
Copy link
Owner

you start working on this problem or is time to delete? (question) thx

Huh? I'm not sure what you mean.

I have confirmed that this problem exists on my instance as well. I suspect this issue has been caused by upgrade of requests library. This integration uses gkeepapi library and it seems to be highly dependent on version of requests.

@BebeMischa
Copy link

So, @PiotrMachowski , do we need to check/update something in our systems, or is it something you would need to take care of?
If we can fix it our self, fine, if we just do know how... ;-)

@PiotrMachowski
Copy link
Owner

No, you can't fix it. I'm not sure if I can fix it, as it seems to be a problem with a library that I use

@BebeMischa
Copy link

Well, then I just wish you the strength to over-win this ;-) Will wait quietly with hope on my mind ;-)

@cagarrinho
Copy link

Same here! @PiotrMachowski
I sincerely hope you can help us. this integration of yours is very important to me. Thank you very much.

@Disturbx
Copy link

I'd be happy if this is gonna be fixed :)

@ostracizado
Copy link

Would it be possible to use the master token, instead?

Seems it was the work around for other custom integrations, such as @leikoilja's ha-google-home.

@PiotrMachowski
Copy link
Owner

PiotrMachowski commented Jun 2, 2023

@ostracizado it should be possible (library documentation)

@Roei639
Copy link

Roei639 commented Jun 3, 2023

I have the same problem, is there a solution?

@biejay
Copy link

biejay commented Jun 5, 2023

Same here

@slake119
Copy link

Is there a hope this is going to be resolved or is the project dead now?

@PiotrMachowski
Copy link
Owner

@slake119 There is a hope, but I don't have much time at this moment.

@BebeMischa
Copy link

So, should I now declare this card not reliable enough and look for a different approach? It now takes too long... ;-)

@AFumi39
Copy link

AFumi39 commented Jul 11, 2023

So, should I now declare this card not reliable enough and look for a different approach? It now takes too long... ;-)

Let me know if you find a different method do manage GKeep lists on HA, please.
Now that Google dismissed 3rd party list integration from Assistant, I had to come back to GKeep and there's no working integration :(

PS: I will abandon any Google product as soon as possible. No improvements, only removed features, lot of EOL, no support, nothing that works well, many bugs, etc

@BebeMischa
Copy link

that's, what I mean. I will just look for completely another solution. Bye bye Google Keep... ;-)

@PiotrMachowski
Copy link
Owner

that's, what I mean. I will just look for completely another solution. Bye bye Google Keep... ;-)

It might be a good idea - this integration uses a library that uses an unofficial way to get data from GKeep, so there's absolutely no guarantee. Additionally it constantly keeps on breaking on changes of underlying libraries/python versions.

@AFumi39
Copy link

AFumi39 commented Jul 11, 2023

Yeah, I use Bring! from few months, right now. Lot better than Keep (but there's also AnyList and similar). But with last changes of Google Assistant, from today custom shopping lists are not supported anymore. You have to use Google Keep, so I'm trying to find a way to connect with Keep and maybe sync it with Bring

@milandzuris
Copy link
Author

Áno, používam Bring! od pár mesiacov práve teraz. Oveľa lepšie ako Keep (ale je tu aj AnyList a podobne). S poslednými zmenami Asistenta Google však od dnešného dňa už nie sú podporované vlastné nákupné zoznamy. Musíte používať službu Google Keep, takže sa snažím nájsť spôsob, ako sa spojiť s aplikáciou Keep a možno ju synchronizovať s aplikáciou Bring

same

@ELind77
Copy link

ELind77 commented Jul 31, 2023

I have the same problem.

Cross-listing the dependency issue ticket for visibility: simon-weber/gpsoauth#48

@CodeNeedsCoffee
Copy link

you start working on this problem or is time to delete? (question) thx

Huh? I'm not sure what you mean.

I have confirmed that this problem exists on my instance as well. I suspect this issue has been caused by upgrade of requests library. This integration uses gkeepapi library and it seems to be highly dependent on version of requests.

What version of requests is your integration dependent on? I'm wondering if it would be possible to maybe have a local version of the requests library for the specific version the integration requires and import it from a local source for this specific integration..?

@PiotrMachowski
Copy link
Owner

At this moment integration uses version that is installed by HA (requests==2.31.0 for HA 2023.7.3)

@elfux
Copy link

elfux commented Aug 7, 2023

so, still not working? thanks!

@superharaldsson
Copy link

Is there any hope of getting it going again? Or should I start looking for other options?

@CodeNeedsCoffee
Copy link

Just wanted to chime in here with my personal "hot take"; I did try to create a specific docker image that individuals could use that would host the correct version of the libraries and rebuild the HA integration to use this new docker, but after reviewing the gkeepapi and eventually getting the docker to work, I realized that there are some security flaws with this approach so I decided to not pursue the project any further. I will be working on a new one since I'm still determined to integrate Google Keep with Home Assistant, but I do not believe this specific component will be coming back to life in the near future.

@PiotrMachowski
Copy link
Owner

@CodeNeedsCoffee what have you changed to make it work?

@jgkawell
Copy link

jgkawell commented Nov 4, 2023

Any update on this? I have the same use case as others where I wish to use Google Assistant to maintain grocery lists in Keep and sync with Home Assistant. Is there a path forward with the current integration? If someone has a path in the right direction I'm open to attempting a fix.

@CodeNeedsCoffee
Copy link

CodeNeedsCoffee commented Dec 4, 2023 via email

@StefanoGiu
Copy link

I used keyring and tokens to make it work again. How can I create a pull request?

@tjorim
Copy link

tjorim commented Feb 14, 2024

Feel free to Google how to use git/GitHub. Basically fork this repo, make the code adjustments, push it and open a PR.

@StefanoGiu
Copy link

Done

@PiotrMachowski
Copy link
Owner

Thanks @StefanoGiu, I'll check it out 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests