Skip to content

Commit c77bd1a

Browse files
authored
Make sure offline sets associated flags (#12718)
1 parent 2fa6d03 commit c77bd1a

File tree

1 file changed

+13
-2
lines changed

1 file changed

+13
-2
lines changed

rust/src/main.rs

+13-2
Original file line numberDiff line numberDiff line change
@@ -157,17 +157,28 @@ fn main() {
157157
flush_and_exit(DATAERR, &log);
158158
};
159159

160-
selenium_manager.set_logger(log);
161160
selenium_manager.set_browser_version(cli.browser_version.unwrap_or_default());
162161
selenium_manager.set_driver_version(cli.driver_version.unwrap_or_default());
163162
selenium_manager.set_browser_path(cli.browser_path.unwrap_or_default());
164163
selenium_manager.set_os(cli.os.unwrap_or_default());
165164
selenium_manager.set_arch(cli.arch.unwrap_or_default());
166165
selenium_manager.set_ttl(cli.ttl);
167-
selenium_manager.set_offline(cli.offline);
168166
selenium_manager.set_force_browser_download(cli.force_browser_download);
169167
selenium_manager.set_avoid_browser_download(cli.avoid_browser_download);
170168
selenium_manager.set_cache_path(cache_path.clone());
169+
selenium_manager.set_offline(cli.offline);
170+
if cli.offline {
171+
if cli.force_browser_download {
172+
log.warn("Offline flag set, but also asked to force downloads. Honouring offline flag");
173+
}
174+
selenium_manager.set_force_browser_download(false);
175+
if !cli.avoid_browser_download {
176+
log.warn("Offline flag set, but also asked not to avoid browser downloads. Honouring offline flag");
177+
}
178+
selenium_manager.set_avoid_browser_download(true);
179+
}
180+
selenium_manager.set_logger(log);
181+
171182

172183
if cli.clear_cache || BooleanKey("clear-cache", false).get_value() {
173184
clear_cache(selenium_manager.get_logger(), &cache_path);

0 commit comments

Comments
 (0)