Skip to content

Commit

Permalink
use eq_ignore_ascii_case
Browse files Browse the repository at this point in the history
Signed-off-by: Zahari Dichev <[email protected]>
  • Loading branch information
zaharidichev committed Jan 14, 2024
1 parent 0ff0fc2 commit be89206
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion linkerd/app/src/env.rs
Original file line number Diff line number Diff line change
Expand Up @@ -818,7 +818,9 @@ pub fn parse_config<S: Strings>(strings: &S) -> Result<super::Config, EnvError>
Some(socket) => match &tls.id {

Check warning on line 818 in linkerd/app/src/env.rs

View check run for this annotation

Codecov / codecov/patch

linkerd/app/src/env.rs#L818

Added line #L818 was not covered by tests
// TODO: perform stricter SPIFFE ID validation following:
// https://github.com/spiffe/spiffe/blob/27b59b81ba8c56885ac5d4be73b35b9b3305fd7a/standards/SPIFFE-ID.md
identity::Id::Uri(uri) if uri.scheme() == SPIFFE_ID_URI_SCHEME => {
identity::Id::Uri(uri)
if uri.scheme().eq_ignore_ascii_case(SPIFFE_ID_URI_SCHEME) =>

Check warning on line 822 in linkerd/app/src/env.rs

View check run for this annotation

Codecov / codecov/patch

linkerd/app/src/env.rs#L821-L822

Added lines #L821 - L822 were not covered by tests
{
identity::Config::Spire {
tls,
client: spire::Config {

Check warning on line 826 in linkerd/app/src/env.rs

View check run for this annotation

Codecov / codecov/patch

linkerd/app/src/env.rs#L826

Added line #L826 was not covered by tests
Expand Down

0 comments on commit be89206

Please sign in to comment.