Cookie improvements #166

sinun98 · 2021-05-26T21:55:34Z

Kann man adden oder auch nicht

sinun98 · 2021-05-26T22:17:29Z

tools/its.py

+        # prüfen, ob Cookies gesetzt wurden und in Session übernehmen
+        try:
+            cookie = driver.get_cookie("bm_sz")
+            time.sleep(10)


Sleep muss noch raus

Floskinner · 2021-05-27T08:57:28Z

Vielleicht hier noch ergänzend zu dem Thema...:

Was man auch noch testen könnte, dass man bei Zeile 256 in der Funktion driver_enter_code

# Maus-Bewegung hinzufügen (nicht sichtbar)       
action.move_by_offset(10, 20).perform()

hier noch weiteres klicken mit pause einzufügen kann

# Maus-Bewegung hinzufügen (nicht sichtbar)       
action.move_by_offset(10, 20).perform()
time.sleep(1)
action.move_by_offset(1, 2).perform()
time.sleep(2)
action.move_by_offset(2,1).perform()

Hatte damals die Erfahrung mit meinem Bot gemacht, dass bereits 2 mal i wo klicken viel gebracht hat bezüglich dem Bot-Schutz (aktuell sind es hier ja nur 1 klick).
Die Idee, dass der User wirklich selbst wo klicken kann ist natürlich auch nicht schlecht und schadet an sich ja nicht

ralfescher · 2021-05-27T09:53:44Z

ich habe gerade die Änderungen ausprobiert
bei mir kommt jetzt [err] [renew_cookies_code] WebDriverException exception raised - retry 1
und endet mit
2021-05-27 11:50:42.173 impfterminservice [err] [renew_cookies_code] WebDriverException exception raised - retry 10
2021-05-27 11:50:42.177 impfterminservice [err] [*TER1 | 70376] (unknown error: DevToolsActivePort file doesn't exist)
(The process started from chrome location /usr/bin/chromium-browser is no longer running, so ChromeDriver is assuming that Chrome has crashed.)

sinun98 · 2021-05-27T10:30:18Z

ich habe gerade die Änderungen ausprobiert
bei mir kommt jetzt [err] [renew_cookies_code] WebDriverException exception raised - retry 1
und endet mit
2021-05-27 11:50:42.173 impfterminservice [err] [renew_cookies_code] WebDriverException exception raised - retry 10
2021-05-27 11:50:42.177 impfterminservice [err] [*TER1 | 70376] (unknown error: DevToolsActivePort file doesn't exist)
(The process started from chrome location /usr/bin/chromium-browser is no longer running, so ChromeDriver is assuming that Chrome has crashed.)

Falls du das in meinem Branch ausprobiert hast liegt es vermutlich daran das alte Chrome Driver darin sind.

sinun98 · 2021-05-27T10:33:09Z

Vielleicht hier noch ergänzend zu dem Thema...:

Was man auch noch testen könnte, dass man bei Zeile 256 in der Funktion driver_enter_code
# Maus-Bewegung hinzufügen (nicht sichtbar)       
action.move_by_offset(10, 20).perform()
hier noch weiteres klicken mit pause einzufügen kann
# Maus-Bewegung hinzufügen (nicht sichtbar)       
action.move_by_offset(10, 20).perform()
time.sleep(1)
action.move_by_offset(1, 2).perform()
time.sleep(2)
action.move_by_offset(2,1).perform()
Hatte damals die Erfahrung mit meinem Bot gemacht, dass bereits 2 mal i wo klicken viel gebracht hat bezüglich dem Bot-Schutz (aktuell sind es hier ja nur 1 klick).
Die Idee, dass der User wirklich selbst wo klicken kann ist natürlich auch nicht schlecht und schadet an sich ja nicht

Da hast du recht. Hab nur die Befürchtung dass es wieder geblockt wird wenn immer genau dasselbe Verhalten abgespielt wird.

ralfescher · 2021-05-27T10:42:11Z

ich habe gerade die Änderungen ausprobiert
bei mir kommt jetzt [err] [renew_cookies_code] WebDriverException exception raised - retry 1
und endet mit
2021-05-27 11:50:42.173 impfterminservice [err] [renew_cookies_code] WebDriverException exception raised - retry 10
2021-05-27 11:50:42.177 impfterminservice [err] [*TER1 | 70376] (unknown error: DevToolsActivePort file doesn't exist)
(The process started from chrome location /usr/bin/chromium-browser is no longer running, so ChromeDriver is assuming that Chrome has crashed.)

Falls du das in meinem Branch ausprobiert hast liegt es vermutlich daran das alte Chrome Driver darin sind.

Ich arbeite normalerweise nicht mit GitHub, ich habe Deinen Branch wie folgt auf den original Beta Branch angewendet:
(das Thema ist hier zwar off-topic, aber ich denke das ist innerhalb der Pull-Request Diskussion okay)

ralf@raspberrypi4:~/vaccipy $ git checkout beta
Branch 'beta' set up to track remote branch 'beta' from 'origin'.
Switched to a new branch 'beta'
ralf@raspberrypi4:~/vaccipy $ git checkout -b beta-cookie
Switched to a new branch 'beta-cookie'
ralf@raspberrypi4:~/vaccipy $ git pull 'https://github.com/sinun98/vaccipy.git' cookieimprovements
From https://github.com/sinun98/vaccipy
 * branch            cookieimprovements -> FETCH_HEAD
Auto-merging tools/its.py
Merge made by the 'recursive' strategy.
 main.py      |  2 +-
 tools/its.py | 32 ++++++++++++++++++++++++++++++++

ralfescher · 2021-05-27T10:45:15Z

Die Idee, dass der User wirklich selbst wo klicken kann ist natürlich auch nicht schlecht und schadet an sich ja nicht
Da hast du recht. Hab nur die Befürchtung dass es wieder geblockt wird wenn immer genau dasselbe Verhalten abgespielt wird.

Warum nicht die statischen Werte durch Random Werte ersetzen?
(betrifft auch Anzahl der Klicks und die Pausen)

ralfescher · 2021-05-27T11:02:51Z

tools/its.py

-                    'Cookie': cookies
-                })
-
+                    "Es die Cookies müssen manuell im Browser generiert werden.\n")


sinun98 · 2021-05-27T11:03:11Z

Hab die Vorschläge mit random Werten hinzugefügt. Klappt bei mir sehr gut, allerdings nicht headless. Schlägt die automatisierte Generation fehl wird der Nutzer aufgefordert manuell zu Klicken.

timreibe · 2021-05-27T14:00:55Z

Die Idee, dass der User wirklich selbst wo klicken kann ist natürlich auch nicht schlecht und schadet an sich ja nicht
Da hast du recht. Hab nur die Befürchtung dass es wieder geblockt wird wenn immer genau dasselbe Verhalten abgespielt wird.

Warum nicht die statischen Werte durch Random Werte ersetzen?
(betrifft auch Anzahl der Klicks und die Pausen)

@sinun98 kannst du das noch einbauen? geht bestimmt auch eine weitere Range, von 0-500 oder so

edit: sehe grad du hasts schon, vllt noch range anpassen. Wenn wir von Pixeln reden sind 1-9 Pixel nicht so viel haha

sinun98 · 2021-05-27T14:31:30Z

Die Idee, dass der User wirklich selbst wo klicken kann ist natürlich auch nicht schlecht und schadet an sich ja nicht
Da hast du recht. Hab nur die Befürchtung dass es wieder geblockt wird wenn immer genau dasselbe Verhalten abgespielt wird.

Warum nicht die statischen Werte durch Random Werte ersetzen?
(betrifft auch Anzahl der Klicks und die Pausen)

@sinun98 kannst du das noch einbauen? geht bestimmt auch eine weitere Range, von 0-500 oder so

edit: sehe grad du hasts schon, vllt noch range anpassen. Wenn wir von Pixeln reden sind 1-9 Pixel nicht so viel haha

Done

Floskinner · 2021-05-27T15:00:32Z

tools/its.py

@@ -255,9 +255,15 @@ def driver_enter_code(self, driver, plz_impfzentrum):
        # Maus-Bewegung hinzufügen (nicht sichtbar)
        action.move_by_offset(10, 20).perform()
        time.sleep(randint(1, 3))
-        action.move_by_offset(randint(1, 9), randint(1, 9)).perform()
+        try:
+            action.move_by_offset(randint(1, 100), randint(1, 100)).perform()


Man muss halt aufpassen, dass nicht aus versehen irgendwo auf einen Link geklickt wird

Wäre ja nicht schlimm oder?

ah ne stimmt, da hatte ich einen Denkfehler, die Cookies bleiben ja erhalten, selbst wenn man auf die Startseite weitergeleitet wird ^^

Jedoch hätte ich evtl. noch einen anderen Verbesserungsvorschlag
Du könntest das ganze ja in einer Schleife machen, dann bräuchtest du nicht so viele Zeilen und nur 1 try / except

for i in range(3): try: action.move_by_offset(randint(1, 100), randint(1, 100)).perform() time.sleep(randint(1, 3)) except Exception as error: #evt fehler in den Logs speichern, damit man weiß was los war pass

Cookie improvements

26b429e

sinun98 commented May 26, 2021

View reviewed changes

cookie improvements+manual mode

b73c039

ralfescher reviewed May 27, 2021

View reviewed changes

typo

c16d706

higher random values

34b01ef

Floskinner reviewed May 27, 2021

View reviewed changes

sinun98 and others added 2 commits May 27, 2021 18:14

code reduce

bcc04fd

Merge branch 'beta' into cookieimprovements

d4958a4

sinun98 merged commit 45e8e0f into timreibe:beta May 27, 2021

sinun98 deleted the cookieimprovements branch May 28, 2021 10:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Cookie improvements #166

Cookie improvements #166

sinun98 commented May 26, 2021

sinun98 May 26, 2021

Floskinner commented May 27, 2021

ralfescher commented May 27, 2021

sinun98 commented May 27, 2021

sinun98 commented May 27, 2021

ralfescher commented May 27, 2021 •

edited

Loading

ralfescher commented May 27, 2021

ralfescher May 27, 2021

sinun98 May 27, 2021

sinun98 commented May 27, 2021 •

edited

Loading

timreibe commented May 27, 2021 •

edited

Loading

sinun98 commented May 27, 2021

Floskinner May 27, 2021

sinun98 May 27, 2021

Floskinner May 27, 2021

Cookie improvements #166

Cookie improvements #166

Conversation

sinun98 commented May 26, 2021

sinun98 May 26, 2021

Choose a reason for hiding this comment

Floskinner commented May 27, 2021

ralfescher commented May 27, 2021

sinun98 commented May 27, 2021

sinun98 commented May 27, 2021

ralfescher commented May 27, 2021 • edited Loading

ralfescher commented May 27, 2021

ralfescher May 27, 2021

Choose a reason for hiding this comment

sinun98 May 27, 2021

Choose a reason for hiding this comment

sinun98 commented May 27, 2021 • edited Loading

timreibe commented May 27, 2021 • edited Loading

sinun98 commented May 27, 2021

Floskinner May 27, 2021

Choose a reason for hiding this comment

sinun98 May 27, 2021

Choose a reason for hiding this comment

Floskinner May 27, 2021

Choose a reason for hiding this comment

ralfescher commented May 27, 2021 •

edited

Loading

sinun98 commented May 27, 2021 •

edited

Loading

timreibe commented May 27, 2021 •

edited

Loading