loblab / noip-renew

Auto renew (confirm) noip.com free hosts
Apache License 2.0
423 stars 134 forks source link

Unable to click on "Confirm" (UI Update?) #104

Open rubenvg95 opened 5 months ago

rubenvg95 commented 5 months ago

Hello,

I got this error today:

`2024/01/14 11:05:19] - Debug level: 2 [2024/01/14 11:05:19] - Opening https://www.noip.com/login... [2024/01/14 11:05:21] - Logging in... [2024/01/14 11:05:25] - Opening https://my.noip.com/dynamic-dns... [2024/01/14 11:05:28] - Message: no such element: Unable to locate element: {"method":"xpath","selector":".//following-sibling::td[4]/button[contains(@class, 'btn')]"} (Session info: headless chrome=90.0.4430.212) Stacktrace:

0 0x560e0bb1e7f9

1 0x560e0babe3b3

2 0x560e0b806016

3 0x560e0b83a81e

4 0x560e0b831e67

5 0x560e0b85dded

6 0x560e0b831d7b

7 0x560e0b85dfce

8 0x560e0b86e9e1

9 0x560e0b85dc93

10 0x560e0b82fce4

11 0x560e0b8314d2

12 0x560e0baea542

13 0x560e0baf9ce7

14 0x560e0baf99e4

15 0x560e0bafe13a

16 0x560e0bafa5b9

17 0x560e0badfe00

18 0x560e0bb115d2

19 0x560e0bb11778 <unkno...`

It seems to be related with: def get_host_button(host, iteration): return host.find_element(By.XPATH, ".//following-sibling::td[4]/button[contains(@class, 'btn')]")

christoschronopoulos commented 4 months ago

I have exactly the same problem. I checked the button element class on Chrome and then changed the python function (see below) but with no success. It doesn't work.

def get_host_button(host, iteration): return host.find_element(By.XPATH, ".//following-sibling::td[4]/button[contains(@class, 'btn btn-labeled btn-success')]")

christoschronopoulos commented 4 months ago

I solved it by changing the noip-renew.py get_host_button function to the following:

def get_host_button(host, iteration): return host.find_element(By.XPATH, "//td[6]/button[contains(@class, 'btn-success')]")

christoschronopoulos commented 4 months ago

thanks can you share your fork

Unfortunately, I haven't published a fork of the project. However, the aforementioned code snippet is the only change required in the code.

EDIT: I just created a fork and made the update to the file. Please let me know if it works. https://github.com/christoschronopoulos/noip-renew/tree/master