bug: remove dbg!() (#292)

Closes #287
mozilla-services · Oct 28, 2021 · 1eca808 · 1eca808
1 parent 2381866
commit 1eca808
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/.cargo/audit.toml b/.cargo/audit.toml
@@ -4,4 +4,6 @@ ignore = [
     "RUSTSEC-2021-0078", # Bound by Sentry 0.19 requiring Hyper 0.13
     "RUSTSEC-2021-0079", # Bound by Sentry 0.19 requiring Hyper 0.13
     "RUSTSEC-2021-0115", # Bound by rusoto 0.45 & fernet 0.1.4 (not a risk)
+    "RUSTSEC-2020-0159", # Bound by time
+    "RUSTSEC-2020-0071", # Bound by sentry, cadence, etc.
 ]
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -85,7 +85,9 @@ jobs:
             cargo build
       - run:
           name: Check formatting
-          command: cargo fmt -- --check
+          command: |
+            cargo fmt -- --check
+            cargo clippy --all --all-targets --all-features -- -D warnings --deny=clippy::dbg_macro
       - run:
           name: Integration tests
           command: py.test -v

diff --git a/autoendpoint/src/extractors/authorization_check.rs b/autoendpoint/src/extractors/authorization_check.rs
@@ -35,7 +35,7 @@ impl AuthorizationCheck {
                 sign_with_key(key.as_bytes(), uaid.to_simple().to_string().as_bytes())
                     .map_err(ApiErrorKind::RegistrationSecretHash)?;
 
-            dbg!(&expected_token, &token);
+            debug!("expected: {:?}, recv'd {:?}", &expected_token, &token);
             if expected_token.len() == token.len()
                 && openssl::memcmp::eq(expected_token.as_bytes(), token.as_bytes())
             {