From 5050d1d21af23eec5ae522f7f4542a7c26baa061 Mon Sep 17 00:00:00 2001
From: Nicholas Ormrod <njormrod@meta.com>
Date: Fri, 15 Nov 2024 12:35:11 -0800
Subject: [PATCH] facebook-unused-include-check in fbcode/fizz

Summary:
Remove headers flagged by facebook-unused-include-check over fbcode.fizz.

+ format and autodeps

This is a codemod. It was automatically generated and will be landed once it is approved and tests are passing in sandcastle.
You have been added as a reviewer by Sentinel or Butterfly.

Autodiff project: uif
Autodiff partition: fbcode.fizz
Autodiff bookmark: ad.uif.fbcode.fizz

Reviewed By: dtolnay

Differential Revision: D65957922

fbshipit-source-id: fc5fa26e566ce1a78c2d30e3c47de2081f855756
---
 fizz/backend/BUCK                                             | 1 -
 fizz/backend/openssl/crypto/aead/OpenSSLEVPCipher.cpp         | 1 -
 fizz/backend/openssl/crypto/signature/Signature.cpp           | 2 --
 fizz/backend/openssl/crypto/test/BUCK                         | 1 -
 fizz/backend/openssl/crypto/test/OpenSSLKeyUtilsTest.cpp      | 2 --
 fizz/crypto/hpke/test/BUCK                                    | 2 --
 fizz/crypto/hpke/test/ContextTest.cpp                         | 2 --
 fizz/experimental/batcher/test/BUCK                           | 1 -
 fizz/experimental/batcher/test/BatcherTest.cpp                | 1 -
 fizz/extensions/delegatedcred/BUCK                            | 4 ----
 .../delegatedcred/DelegatedCredentialCertManager.cpp          | 1 -
 fizz/extensions/delegatedcred/Types.cpp                       | 1 -
 fizz/extensions/tokenbinding/Validator.cpp                    | 1 -
 fizz/extensions/tokenbinding/test/BUCK                        | 1 -
 .../tokenbinding/test/TokenBindingServerExtensionTest.cpp     | 1 -
 fizz/record/test/BUCK                                         | 1 -
 fizz/record/test/ExtensionsTest.cpp                           | 1 -
 fizz/server/BUCK                                              | 2 --
 fizz/server/SlidingBloomReplayCache.cpp                       | 4 ----
 fizz/server/test/AeadTicketCipherTest.cpp                     | 1 -
 fizz/server/test/BUCK                                         | 2 --
 fizz/server/test/TicketPolicyTest.cpp                         | 1 -
 fizz/test/BUCK                                                | 1 -
 fizz/test/BogoShim.cpp                                        | 1 -
 fizz/tool/FizzClientLoadGenCommand.cpp                        | 2 --
 fizz/tool/FizzServerBenchmarkCommand.cpp                      | 3 ---
 26 files changed, 41 deletions(-)

diff --git a/fizz/backend/BUCK b/fizz/backend/BUCK
index 2f13ad7440f..244b30d862a 100644
--- a/fizz/backend/BUCK
+++ b/fizz/backend/BUCK
@@ -76,7 +76,6 @@ cpp_library(
     ],
     deps = [
         "//fizz/crypto/aead:cryptoutil",
-        "//folly:scope_guard",
         "//folly/lang:checked_math",
     ],
     exported_deps = [
diff --git a/fizz/backend/openssl/crypto/aead/OpenSSLEVPCipher.cpp b/fizz/backend/openssl/crypto/aead/OpenSSLEVPCipher.cpp
index 9e2dbfa8df7..922c309d4da 100644
--- a/fizz/backend/openssl/crypto/aead/OpenSSLEVPCipher.cpp
+++ b/fizz/backend/openssl/crypto/aead/OpenSSLEVPCipher.cpp
@@ -9,7 +9,6 @@
 #include <fizz/backend/openssl/crypto/aead/OpenSSLEVPCipher.h>
 #include <fizz/crypto/aead/CryptoUtil.h>
 #include <folly/lang/CheckedMath.h>
-#include <functional>
 
 namespace fizz {
 namespace openssl {
diff --git a/fizz/backend/openssl/crypto/signature/Signature.cpp b/fizz/backend/openssl/crypto/signature/Signature.cpp
index e72a1e865d8..295dbea1190 100644
--- a/fizz/backend/openssl/crypto/signature/Signature.cpp
+++ b/fizz/backend/openssl/crypto/signature/Signature.cpp
@@ -8,10 +8,8 @@
 
 #include <fizz/backend/openssl/crypto/OpenSSLKeyUtils.h>
 #include <fizz/backend/openssl/crypto/signature/Signature.h>
-#include <openssl/crypto.h>
 
 #include <folly/Conv.h>
-#include <folly/ScopeGuard.h>
 #include <folly/ssl/OpenSSLPtrTypes.h>
 
 using namespace folly;
diff --git a/fizz/backend/openssl/crypto/test/BUCK b/fizz/backend/openssl/crypto/test/BUCK
index 508439df547..76be47caa66 100644
--- a/fizz/backend/openssl/crypto/test/BUCK
+++ b/fizz/backend/openssl/crypto/test/BUCK
@@ -22,7 +22,6 @@ cpp_unittest(
     deps = [
         "//fizz/backend:openssl",
         "//fizz/crypto/test:TestUtil",
-        "//folly:string",
         "//folly/portability:gtest",
     ],
 )
diff --git a/fizz/backend/openssl/crypto/test/OpenSSLKeyUtilsTest.cpp b/fizz/backend/openssl/crypto/test/OpenSSLKeyUtilsTest.cpp
index 63ddcf44f17..acf4cba1e29 100644
--- a/fizz/backend/openssl/crypto/test/OpenSSLKeyUtilsTest.cpp
+++ b/fizz/backend/openssl/crypto/test/OpenSSLKeyUtilsTest.cpp
@@ -11,8 +11,6 @@
 #include <fizz/backend/openssl/crypto/OpenSSLKeyUtils.h>
 #include <fizz/crypto/test/TestUtil.h>
 
-#include <folly/String.h>
-
 namespace fizz {
 using namespace test;
 
diff --git a/fizz/crypto/hpke/test/BUCK b/fizz/crypto/hpke/test/BUCK
index 6712fcdd192..00604cbdd15 100644
--- a/fizz/crypto/hpke/test/BUCK
+++ b/fizz/crypto/hpke/test/BUCK
@@ -25,13 +25,11 @@ cpp_unittest(
     ],
     supports_static_listing = False,
     deps = [
-        ":mocks",
         "//fizz/backend:openssl",
         "//fizz/crypto/aead/test:test_util",
         "//fizz/crypto/hpke:context",
         "//fizz/crypto/hpke:utils",
         "//fizz/crypto/test:TestUtil",
-        "//fizz/protocol:types",
         "//fizz/record:record",
     ],
 )
diff --git a/fizz/crypto/hpke/test/ContextTest.cpp b/fizz/crypto/hpke/test/ContextTest.cpp
index 3b003eb0362..d0d1f363db8 100644
--- a/fizz/crypto/hpke/test/ContextTest.cpp
+++ b/fizz/crypto/hpke/test/ContextTest.cpp
@@ -13,9 +13,7 @@
 #include <fizz/crypto/aead/test/TestUtil.h>
 #include <fizz/crypto/hpke/Context.h>
 #include <fizz/crypto/hpke/Utils.h>
-#include <fizz/crypto/hpke/test/Mocks.h>
 #include <fizz/crypto/test/TestUtil.h>
-#include <fizz/protocol/Types.h>
 #include <fizz/record/Types.h>
 
 using namespace fizz::test;
diff --git a/fizz/experimental/batcher/test/BUCK b/fizz/experimental/batcher/test/BUCK
index b0791cb0d15..dc0477f019f 100644
--- a/fizz/experimental/batcher/test/BUCK
+++ b/fizz/experimental/batcher/test/BUCK
@@ -13,7 +13,6 @@ cpp_unittest(
         "//fizz/experimental/batcher:batcher",
         "//fizz/experimental/client:batch_signature_peer_cert",
         "//fizz/experimental/server:batch_signature_async_self_cert",
-        "//fizz/server:protocol",
         "//fizz/server/test:mocks",
         "//folly/portability:gtest",
     ],
diff --git a/fizz/experimental/batcher/test/BatcherTest.cpp b/fizz/experimental/batcher/test/BatcherTest.cpp
index fe15b3f2536..55823213c62 100644
--- a/fizz/experimental/batcher/test/BatcherTest.cpp
+++ b/fizz/experimental/batcher/test/BatcherTest.cpp
@@ -12,7 +12,6 @@
 #include <fizz/experimental/batcher/Batcher.h>
 #include <fizz/experimental/client/BatchSignaturePeerCert.h>
 #include <fizz/experimental/server/BatchSignatureAsyncSelfCert.h>
-#include <fizz/server/State.h>
 #include <fizz/server/test/Mocks.h>
 #include <folly/portability/GTest.h>
 
diff --git a/fizz/extensions/delegatedcred/BUCK b/fizz/extensions/delegatedcred/BUCK
index 3cada898b96..cab88be7a2d 100644
--- a/fizz/extensions/delegatedcred/BUCK
+++ b/fizz/extensions/delegatedcred/BUCK
@@ -11,7 +11,6 @@ cpp_library(
         "Types.h",
     ],
     deps = [
-        "//folly:string",
         "//folly/io:iobuf",
     ],
     exported_deps = [
@@ -129,9 +128,6 @@ cpp_library(
     headers = [
         "DelegatedCredentialCertManager.h",
     ],
-    deps = [
-        ":delegated_credential_utils",
-    ],
     exported_deps = [
         ":delegated_credential",
         ":self_delegated_credential",
diff --git a/fizz/extensions/delegatedcred/DelegatedCredentialCertManager.cpp b/fizz/extensions/delegatedcred/DelegatedCredentialCertManager.cpp
index 4269c39df40..a215d4eb5ff 100644
--- a/fizz/extensions/delegatedcred/DelegatedCredentialCertManager.cpp
+++ b/fizz/extensions/delegatedcred/DelegatedCredentialCertManager.cpp
@@ -6,7 +6,6 @@
  *  LICENSE file in the root directory of this source tree.
  */
 #include <fizz/extensions/delegatedcred/DelegatedCredentialCertManager.h>
-#include <fizz/extensions/delegatedcred/DelegatedCredentialUtils.h>
 
 using namespace fizz::server;
 
diff --git a/fizz/extensions/delegatedcred/Types.cpp b/fizz/extensions/delegatedcred/Types.cpp
index 3858fbb735d..1b213e68d22 100644
--- a/fizz/extensions/delegatedcred/Types.cpp
+++ b/fizz/extensions/delegatedcred/Types.cpp
@@ -8,7 +8,6 @@
 #include <fizz/extensions/delegatedcred/Types.h>
 #include <fizz/record/Types.h>
 
-#include <folly/String.h>
 #include <folly/io/Cursor.h>
 
 using namespace fizz::extensions;
diff --git a/fizz/extensions/tokenbinding/Validator.cpp b/fizz/extensions/tokenbinding/Validator.cpp
index ae39d398f3c..636fd2be58b 100644
--- a/fizz/extensions/tokenbinding/Validator.cpp
+++ b/fizz/extensions/tokenbinding/Validator.cpp
@@ -8,7 +8,6 @@
 
 #include <fizz/extensions/tokenbinding/Validator.h>
 
-#include <fizz/backend/openssl/OpenSSL.h>
 #include <fizz/extensions/tokenbinding/Utils.h>
 #include <fizz/protocol/DefaultFactory.h>
 
diff --git a/fizz/extensions/tokenbinding/test/BUCK b/fizz/extensions/tokenbinding/test/BUCK
index d5163d232e3..5c5cb0c0bc3 100644
--- a/fizz/extensions/tokenbinding/test/BUCK
+++ b/fizz/extensions/tokenbinding/test/BUCK
@@ -27,7 +27,6 @@ cpp_unittest(
         "//fizz/extensions/tokenbinding:token_binding_context",
         "//fizz/extensions/tokenbinding:token_binding_server_extension",
         "//fizz/record:record",
-        "//fizz/server:server_extensions",
         "//folly/portability:gmock",
         "//folly/portability:gtest",
     ],
diff --git a/fizz/extensions/tokenbinding/test/TokenBindingServerExtensionTest.cpp b/fizz/extensions/tokenbinding/test/TokenBindingServerExtensionTest.cpp
index de15445c055..cffc8fed522 100644
--- a/fizz/extensions/tokenbinding/test/TokenBindingServerExtensionTest.cpp
+++ b/fizz/extensions/tokenbinding/test/TokenBindingServerExtensionTest.cpp
@@ -13,7 +13,6 @@
 #include <fizz/extensions/tokenbinding/TokenBindingServerExtension.h>
 #include <fizz/extensions/tokenbinding/Types.h>
 #include <fizz/record/Extensions.h>
-#include <fizz/server/ServerExtensions.h>
 
 using namespace testing;
 
diff --git a/fizz/record/test/BUCK b/fizz/record/test/BUCK
index 14872fd0fa1..ccda2b50db2 100644
--- a/fizz/record/test/BUCK
+++ b/fizz/record/test/BUCK
@@ -51,7 +51,6 @@ cpp_unittest(
     deps = [
         ":extension_tests_base",
         "//fizz/record:record",
-        "//folly:string",
         "//folly/portability:gmock",
         "//folly/portability:gtest",
     ],
diff --git a/fizz/record/test/ExtensionsTest.cpp b/fizz/record/test/ExtensionsTest.cpp
index 4e3833cfdcc..b24d1400488 100644
--- a/fizz/record/test/ExtensionsTest.cpp
+++ b/fizz/record/test/ExtensionsTest.cpp
@@ -11,7 +11,6 @@
 
 #include <fizz/record/Extensions.h>
 #include <fizz/record/test/ExtensionTestsBase.h>
-#include <folly/String.h>
 
 using namespace folly;
 
diff --git a/fizz/server/BUCK b/fizz/server/BUCK
index dd097fd2683..d1690fd6756 100644
--- a/fizz/server/BUCK
+++ b/fizz/server/BUCK
@@ -363,9 +363,7 @@ cpp_library(
     ],
     deps = [
         "//fizz/crypto:random",
-        "//folly:conv",
         "//folly:range",
-        "//folly/hash:hash",
         "//folly/io:iobuf",
         "//folly/portability:unistd",
     ],
diff --git a/fizz/server/SlidingBloomReplayCache.cpp b/fizz/server/SlidingBloomReplayCache.cpp
index 35f71af0ed1..db3d0fda88f 100644
--- a/fizz/server/SlidingBloomReplayCache.cpp
+++ b/fizz/server/SlidingBloomReplayCache.cpp
@@ -8,12 +8,8 @@
 
 #include <fizz/server/SlidingBloomReplayCache.h>
 
-#include <sys/types.h>
-
 #include <fstream>
 
-#include <folly/Conv.h>
-#include <folly/hash/Hash.h>
 #include <folly/io/IOBuf.h>
 #include <folly/portability/Unistd.h>
 
diff --git a/fizz/server/test/AeadTicketCipherTest.cpp b/fizz/server/test/AeadTicketCipherTest.cpp
index 7e0d1e8d967..0f0968d82a2 100644
--- a/fizz/server/test/AeadTicketCipherTest.cpp
+++ b/fizz/server/test/AeadTicketCipherTest.cpp
@@ -16,7 +16,6 @@
 #include <fizz/crypto/test/TestUtil.h>
 #include <fizz/protocol/clock/test/Mocks.h>
 #include <fizz/protocol/test/Mocks.h>
-#include <folly/String.h>
 
 using namespace fizz::test;
 using namespace testing;
diff --git a/fizz/server/test/BUCK b/fizz/server/test/BUCK
index feeae85440e..9d939e6ca81 100644
--- a/fizz/server/test/BUCK
+++ b/fizz/server/test/BUCK
@@ -88,7 +88,6 @@ cpp_unittest(
     ],
     deps = [
         "//fizz/crypto/test:TestUtil",
-        "//fizz/crypto/test:mocks",
         "//fizz/protocol/clock/test:mock_clock",
         "//fizz/server:ticket_policy",
         "//folly/portability:gmock",
@@ -108,7 +107,6 @@ cpp_unittest(
         "//fizz/protocol/clock/test:mock_clock",
         "//fizz/protocol/test:mocks",
         "//fizz/server:aead_ticket_cipher",
-        "//folly:string",
         "//folly/portability:gmock",
         "//folly/portability:gtest",
     ],
diff --git a/fizz/server/test/TicketPolicyTest.cpp b/fizz/server/test/TicketPolicyTest.cpp
index 614f7b353e0..d8d45aa53ad 100644
--- a/fizz/server/test/TicketPolicyTest.cpp
+++ b/fizz/server/test/TicketPolicyTest.cpp
@@ -9,7 +9,6 @@
 #include <folly/portability/GMock.h>
 #include <folly/portability/GTest.h>
 
-#include <fizz/crypto/test/Mocks.h>
 #include <fizz/crypto/test/TestUtil.h>
 #include <fizz/protocol/clock/test/Mocks.h>
 #include <fizz/server/TicketPolicy.h>
diff --git a/fizz/test/BUCK b/fizz/test/BUCK
index f3959cd5e76..a5506a91587 100644
--- a/fizz/test/BUCK
+++ b/fizz/test/BUCK
@@ -90,7 +90,6 @@ cpp_binary(
         "//fizz/crypto:utils",
         "//fizz/server:async_fizz_server",
         "//fizz/server:ticket_types",
-        "//folly:string",
         "//folly/io/async:async_ssl_socket",
         "//folly/io/async:ssl_context",
         "//folly/portability:gflags",
diff --git a/fizz/test/BogoShim.cpp b/fizz/test/BogoShim.cpp
index f4b7f76613f..959d111a4da 100644
--- a/fizz/test/BogoShim.cpp
+++ b/fizz/test/BogoShim.cpp
@@ -13,7 +13,6 @@
 #include <fizz/crypto/Utils.h>
 #include <fizz/server/AsyncFizzServer.h>
 #include <fizz/server/TicketTypes.h>
-#include <folly/String.h>
 #include <folly/io/async/AsyncSSLSocket.h>
 #include <folly/io/async/SSLContext.h>
 #include <folly/portability/GFlags.h>
diff --git a/fizz/tool/FizzClientLoadGenCommand.cpp b/fizz/tool/FizzClientLoadGenCommand.cpp
index 9eea6ab74c4..b2427a9c4d0 100644
--- a/fizz/tool/FizzClientLoadGenCommand.cpp
+++ b/fizz/tool/FizzClientLoadGenCommand.cpp
@@ -10,8 +10,6 @@
 #include <fizz/client/PskSerializationUtils.h>
 #include <fizz/experimental/protocol/BatchSignatureFactory.h>
 #include <fizz/tool/FizzCommandCommon.h>
-#include <fizz/util/Parse.h>
-#include <folly/FileUtil.h>
 #include <folly/Format.h>
 #include <folly/executors/IOThreadPoolExecutor.h>
 #include <folly/futures/Future.h>
diff --git a/fizz/tool/FizzServerBenchmarkCommand.cpp b/fizz/tool/FizzServerBenchmarkCommand.cpp
index 07beac97f4a..b73c6156bdf 100644
--- a/fizz/tool/FizzServerBenchmarkCommand.cpp
+++ b/fizz/tool/FizzServerBenchmarkCommand.cpp
@@ -16,10 +16,7 @@
 #include <fizz/server/SlidingBloomReplayCache.h>
 #include <fizz/server/TicketTypes.h>
 #include <fizz/tool/FizzCommandCommon.h>
-#include <fizz/util/KeyLogWriter.h>
-#include <fizz/util/Parse.h>
 
-#include <folly/Format.h>
 #include <folly/executors/IOThreadPoolExecutor.h>
 #include <folly/futures/Future.h>
 #include <folly/io/async/AsyncSSLSocket.h>