feature: add support to config dns client cache size #622
Triggered via pull request
September 19, 2023 07:49
Status
Success
Total duration
1m 58s
Artifacts
–
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
17 warnings
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/sys/unix/uds.rs#L71
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/sys/unix/uds.rs:71:47
|
71 | pub async fn recv_with_fd(&mut self, buf: &mut [u8], fds: &mut [RawFd]) -> io::Result<(usize, usize)> {
| ^^^^^^^^^ help: consider changing to: `&[u8]`
|
= warning: changing this function will impact semver compatibility
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
= note: `#[warn(clippy::needless_pass_by_ref_mut)]` on by default
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/sys/unix/uds.rs#L71
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/sys/unix/uds.rs:71:63
|
71 | pub async fn recv_with_fd(&mut self, buf: &mut [u8], fds: &mut [RawFd]) -> io::Result<(usize, usize)> {
| ^^^^^^^^^^^^ help: consider changing to: `&[RawFd]`
|
= warning: changing this function will impact semver compatibility
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/udp.rs#L191
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/udp.rs:191:42
|
191 | pub async fn batch_send(&self, msgs: &mut [BatchSendMessage<'_>]) -> io::Result<usize> {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider changing to: `&[BatchSendMessage<'_>]`
|
= warning: changing this function will impact semver compatibility
= note: this is cfg-gated and may require further changes
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/udp.rs#L229
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/udp.rs:229:42
|
229 | pub async fn batch_recv(&self, msgs: &mut [BatchRecvMessage<'_>]) -> io::Result<usize> {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider changing to: `&[BatchRecvMessage<'_>]`
|
= warning: changing this function will impact semver compatibility
= note: this is cfg-gated and may require further changes
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
|
unnecessary hashes around raw string literal:
crates/shadowsocks-service/src/acl/mod.rs#L195
warning: unnecessary hashes around raw string literal
--> crates/shadowsocks-service/src/acl/mod.rs:195:17
|
195 | r#"^(?:(?:\((?:\?:)?\^\|\\\.\)|(?:\^\.(?:\+|\*))?\\\.)((?:[\w-]+(?:\\\.)?)+)|\^((?:[\w-]+(?:\\\.)?)+))\$?$"#,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes
= note: `#[warn(clippy::needless_raw_string_hashes)]` on by default
help: remove all the hashes around the literal
|
195 - r#"^(?:(?:\((?:\?:)?\^\|\\\.\)|(?:\^\.(?:\+|\*))?\\\.)((?:[\w-]+(?:\\\.)?)+)|\^((?:[\w-]+(?:\\\.)?)+))\$?$"#,
195 + r"^(?:(?:\((?:\?:)?\^\|\\\.\)|(?:\^\.(?:\+|\*))?\\\.)((?:[\w-]+(?:\\\.)?)+)|\^((?:[\w-]+(?:\\\.)?)+))\$?$",
|
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks-service/src/local/dns/upstream.rs#L161
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks-service/src/local/dns/upstream.rs:161:62
|
161 | fn check_peekable<F: std::os::unix::io::AsRawFd>(fd: &mut F) -> bool {
| ^^^^^^ help: consider changing to: `&F`
|
= note: this is cfg-gated and may require further changes
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
= note: `#[warn(clippy::needless_pass_by_ref_mut)]` on by default
|
this OR pattern can be rewritten using a range:
src/logging/mod.rs#L58
warning: this OR pattern can be rewritten using a range
--> src/logging/mod.rs:58:9
|
58 | 0 | 1 | 2 | 3 => logging_builder
| ^^^^^^^^^^^^^ help: try: `0..=3`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_patterns
= note: `#[warn(clippy::manual_range_patterns)]` on by default
|
useless conversion to the same type: `u32`:
src/sys.rs#L124
warning: useless conversion to the same type: `u32`
--> src/sys.rs:124:42
|
124 | if libc::initgroups(pwd.pw_name, pwd.pw_gid.try_into().unwrap()) != 0 {
| ^^^^^^^^^^^^^^^^^^^^^
|
= help: consider removing `.try_into()`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
= note: `#[warn(clippy::useless_conversion)]` on by default
|
unneeded `return` statement:
bin/ssurl.rs#L161
warning: unneeded `return` statement
--> bin/ssurl.rs:161:5
|
161 | return ExitCode::FAILURE;
| ^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
= note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
|
161 - return ExitCode::FAILURE;
161 + ExitCode::FAILURE
|
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/sys/unix/uds.rs#L71
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/sys/unix/uds.rs:71:47
|
71 | pub async fn recv_with_fd(&mut self, buf: &mut [u8], fds: &mut [RawFd]) -> io::Result<(usize, usize)> {
| ^^^^^^^^^ help: consider changing to: `&[u8]`
|
= warning: changing this function will impact semver compatibility
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
= note: `#[warn(clippy::needless_pass_by_ref_mut)]` on by default
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/sys/unix/uds.rs#L71
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/sys/unix/uds.rs:71:63
|
71 | pub async fn recv_with_fd(&mut self, buf: &mut [u8], fds: &mut [RawFd]) -> io::Result<(usize, usize)> {
| ^^^^^^^^^^^^ help: consider changing to: `&[RawFd]`
|
= warning: changing this function will impact semver compatibility
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/udp.rs#L191
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/udp.rs:191:42
|
191 | pub async fn batch_send(&self, msgs: &mut [BatchSendMessage<'_>]) -> io::Result<usize> {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider changing to: `&[BatchSendMessage<'_>]`
|
= warning: changing this function will impact semver compatibility
= note: this is cfg-gated and may require further changes
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks/src/net/udp.rs#L229
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks/src/net/udp.rs:229:42
|
229 | pub async fn batch_recv(&self, msgs: &mut [BatchRecvMessage<'_>]) -> io::Result<usize> {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider changing to: `&[BatchRecvMessage<'_>]`
|
= warning: changing this function will impact semver compatibility
= note: this is cfg-gated and may require further changes
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
|
unnecessary hashes around raw string literal:
crates/shadowsocks-service/src/acl/mod.rs#L195
warning: unnecessary hashes around raw string literal
--> crates/shadowsocks-service/src/acl/mod.rs:195:17
|
195 | r#"^(?:(?:\((?:\?:)?\^\|\\\.\)|(?:\^\.(?:\+|\*))?\\\.)((?:[\w-]+(?:\\\.)?)+)|\^((?:[\w-]+(?:\\\.)?)+))\$?$"#,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes
= note: `#[warn(clippy::needless_raw_string_hashes)]` on by default
help: remove all the hashes around the literal
|
195 - r#"^(?:(?:\((?:\?:)?\^\|\\\.\)|(?:\^\.(?:\+|\*))?\\\.)((?:[\w-]+(?:\\\.)?)+)|\^((?:[\w-]+(?:\\\.)?)+))\$?$"#,
195 + r"^(?:(?:\((?:\?:)?\^\|\\\.\)|(?:\^\.(?:\+|\*))?\\\.)((?:[\w-]+(?:\\\.)?)+)|\^((?:[\w-]+(?:\\\.)?)+))\$?$",
|
|
this argument is a mutable reference, but not used mutably:
crates/shadowsocks-service/src/local/dns/upstream.rs#L161
warning: this argument is a mutable reference, but not used mutably
--> crates/shadowsocks-service/src/local/dns/upstream.rs:161:62
|
161 | fn check_peekable<F: std::os::unix::io::AsRawFd>(fd: &mut F) -> bool {
| ^^^^^^ help: consider changing to: `&F`
|
= note: this is cfg-gated and may require further changes
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut
= note: `#[warn(clippy::needless_pass_by_ref_mut)]` on by default
|
this OR pattern can be rewritten using a range:
src/logging/mod.rs#L58
warning: this OR pattern can be rewritten using a range
--> src/logging/mod.rs:58:9
|
58 | 0 | 1 | 2 | 3 => logging_builder
| ^^^^^^^^^^^^^ help: try: `0..=3`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_patterns
= note: `#[warn(clippy::manual_range_patterns)]` on by default
|
unneeded `return` statement:
bin/ssurl.rs#L161
warning: unneeded `return` statement
--> bin/ssurl.rs:161:5
|
161 | return ExitCode::FAILURE;
| ^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
= note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
|
161 - return ExitCode::FAILURE;
161 + ExitCode::FAILURE
|
|