search

TobseF / impf-bot

💉🤖 Bot for the German "ImpfterminService - 116117"
The Unlicense
151 stars 41 forks source link

Chromedriver sucht nach version 91, was in Debian buster nicht verfügbar ist #86

Open piechade opened 3 years ago

piechade commented 3 years ago

In der Docker Datei wird chromium=90.0.4430.212-1~deb10u1 installiert, was völlig richtig ist. Dabei handelt es um die letzte Version im repo. Aber wenn Sie den Container ausführen, sucht der Treiber-Downloader nach dem Chrome 91 Treiber.

Ich habe es so repariert: WebDriverManager.chromedriver().driverVersion("90.0.4430.212").setup();

sdnr commented 3 years ago

should be WebDriverManager.chromedriver().driverVersion("90.0.4430.24").setup(); i guess

piechade commented 3 years ago

should be WebDriverManager.chromedriver().driverVersion("90.0.4430.24").setup(); i guess

You're absolutely right

Timwun commented 3 years ago

Tbh this WebDriverManager isn't a really good solution. It doesn't support ARM (e.g. for Raspberry Pi's) drivers or it's choosing the right version.

:(

c0rrre commented 3 years ago

is the docker image working for u guys? My docker is always complaining that it cant find the chromdriver

tomacco81 commented 3 years ago

With the following adjustments it works for me: https://github.com/TobseF/impf-bot/issues/86#issuecomment-852974499 (DriverFactory.kt) https://github.com/TobseF/impf-bot/issues/36#issuecomment-855217377 (Dockerfile)

SchuhMichael commented 3 years ago

Works for me, too. I have built the image locally, after I've modified DriverFactory.kt following the advise given in https://github.com/TobseF/impf-bot/issues/86#issuecomment-852974499 Dockerfile unchanged.