Skip to content

Commit

Permalink
feat: allow autoconfig when SOCKS5 is enabled
Browse files Browse the repository at this point in the history
Since HTTP module supports SOCKS5 now,
there is no reason not to request autoconfig XML
and outlook configuration anymore.
  • Loading branch information
link2xt committed Aug 9, 2024
1 parent 4a2ebd0 commit 153188d
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions src/configure.rs
Original file line number Diff line number Diff line change
Expand Up @@ -274,14 +274,7 @@ async fn configure(ctx: &Context, param: &mut LoginParam) -> Result<()> {
} else {
// Try receiving autoconfig
info!(ctx, "no offline autoconfig found");
param_autoconfig = if socks5_enabled {
// Currently we can't do http requests through socks5, to not leak
// the ip, just don't do online autoconfig
info!(ctx, "socks5 enabled, skipping autoconfig");
None
} else {
get_autoconfig(ctx, param, &param_domain).await
}
param_autoconfig = get_autoconfig(ctx, param, &param_domain).await;
}
} else {
param_autoconfig = None;
Expand Down

0 comments on commit 153188d

Please sign in to comment.