Fix chrome profile #289

Uunnamed · 2020-08-17T09:52:09Z

#fixies #120

igrishaev · 2020-08-17T11:39:31Z

src/etaoin/driver.clj

-  (let [default #"(\\|/)Default$"
-        p       (string/replace profile default "")]
-    (set-options-args driver [(format "--user-data-dir=%s" p)])))
+  (let [default       #"(\\|/)Default$"


Мне кажется, этот код нужно отрефакторить. Основной момент в том, чтобы для файловых путей использовать объект java.io.File -- это безопасней и работает везде с учетом особенностей ОС.

Обернуть строковый путь в io/file или (new File)

если имя файла Default, то получить родительский файл (.getParent или как-то так)

потом еще раз поняться на уровень выше

этот уровнеь станет data-dir, а имя профиля -- имя файла (.getFileName или как-то так)

Uunnamed added 2 commits August 17, 2020 12:45

fix setting up chrome profile

c3c37ed

add chrome profile dir

c100370

igrishaev reviewed Aug 17, 2020

View reviewed changes

update

faf173b

Uunnamed merged commit edac32b into master Aug 17, 2020

Uunnamed deleted the fix-chrome-profile branch August 17, 2020 12:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix chrome profile #289

Fix chrome profile #289

Uunnamed commented Aug 17, 2020

igrishaev Aug 17, 2020 •

edited

Loading

Fix chrome profile #289

Fix chrome profile #289

Conversation

Uunnamed commented Aug 17, 2020

igrishaev Aug 17, 2020 • edited Loading

Choose a reason for hiding this comment

igrishaev Aug 17, 2020 •

edited

Loading