mirror of
https://github.com/kidfromjupiter/nearby.git
synced 2026-09-16 15:36:12 -04:00
Anay Wadhera
f26d25ed01
The `RandBytes` functions were being placed in the `crypto` namespace, which collides with Chromium's namespace of the same name. Within, `RandBytes` was defined with almost-the-same API. Then in Chromium builds, the Chromium header would be used instead (though somewhat inconsistently). This creates a lot of pain for Chromium development as there's a third-party repository directly depending on headers from Chromium's source tree, and is against the third-party policies for that reason. There are a number of other headers that mirror Chromium and are swapped out in the Chromium build that will cause similar pain, such as: ``` include "crypto/aead.h" include "crypto/ec_private_key.h" include "crypto/hkdf.h" ``` This CL provides a template for how to get rid of these header swaps and give a platform abstraction in nearby instead. We provide a platform abstraction in `platform/crypto.h` (really in `platform/implementation/crypto.h`) which is implemented in `platform/implementation/shared/crypto.cc`. However that implementation is removed by `#ifdef` when in the Chromium build. Then, in the Chromium repo, we will add (separately) an implementation of the same abstraction in `//third_party/nearby/platform_impl` with GN rules to include it in the build. It will replace the implementation from the nearby repo. Copybara import of the project: -- 6ca8099 by danakj <danakj@chromium.org>: Provide a platform abstraction for RandBytes instead of swapping headers The `RandBytes` functions were being placed in the `crypto` namespace, which collides with Chromium's namespace of the same name. Within, `RandBytes` was defined with almost-the-same API. Then in Chromium builds, the Chromium header would be used instead (though somewhat inconsistently). This creates a lot of pain for Chromium development as there's a third-party repository directly depending on headers from Chromium's source tree, and is against the third-party policies for that reason. There are a number of other headers that mirror Chromium and are swapped out in the Chromium build that will cause similar pain, such as: ``` include "crypto/aead.h" include "crypto/ec_private_key.h" include "crypto/hkdf.h" ``` This CL provides a template for how to get rid of these header swaps and give a platform abstraction in nearby instead. We provide a platform abstraction in `platform/crypto.h` (really in `platform/implementation/crypto.h`) which is implemented in `platform/implementation/shared/crypto.cc`. However that implementation is removed by `#ifdef` when in the Chromium build. Then, in the Chromium repo, we will add (separately) an implementation of the same abstraction in `//third_party/nearby/platform_impl` with GN rules to include it in the build. It will replace the implementation from the nearby repo. -- 9c2654b by danakj <danakj@chromium.org>: Remove CryptoSpan, use absl::Span The header swapping of Chromium crypto libraries is problematic, but absl::Span will convert to base::span so there's no need for the typedef even without removing the header swapping yet. -- df1135d by danakj <danakj@chromium.org>: Add missing files -- ab18a15 by danakj <danakj@chromium.org>: Remove the random_unittest.cc from Swift build The file moved, so the Swift package needs its path updated. -- 2038f78 by danakj <danakj@chromium.org>: Combine crypto unittests into crypto_test.cc -- f7ad176 by danakj <danakj@chromium.org>: Add stdint and stddef includes for uint8_t and size_t -- 9f04590 by danakj <danakj@chromium.org>: Mark the shared crypto implementation compatable_with non_prod -- 18eeafd by danakj <danakj@chromium.org>: Add IWYU pragma for crypto implementation PiperOrigin-RevId: 632150866
115 lines
3.4 KiB
Python
115 lines
3.4 KiB
Python
licenses(["notice"])
|
|
|
|
# Copyright 2023 Google LLC
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# https://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
package(default_visibility = [
|
|
"//:__subpackages__",
|
|
"//location/nearby/cpp/sharing/implementation:__subpackages__",
|
|
])
|
|
|
|
licenses(["notice"])
|
|
|
|
# /**********************************
|
|
# * WARNING -- DO NOT MODIFY *
|
|
# **********************************/
|
|
# Only add things to this directory that keep it in sync with Chromium's crypto library found at
|
|
# https://source.chromium.org/chromium/chromium/src/+/main:crypto/
|
|
# New classes that don't originate from there should be added to
|
|
# google3/third_party/nearby/internal/crypto instead.
|
|
cc_library(
|
|
name = "crypto_cros",
|
|
srcs = [
|
|
"aead.cc",
|
|
"ec_private_key.cc",
|
|
"ec_signature_creator.cc",
|
|
"ec_signature_creator_impl.cc",
|
|
"encryptor.cc",
|
|
"hkdf.cc",
|
|
"hmac.cc",
|
|
"nearby_base.cc",
|
|
"openssl_util.cc",
|
|
"rsa_private_key.cc",
|
|
"secure_hash.cc",
|
|
"secure_util.cc",
|
|
"sha2.cc",
|
|
"signature_verifier.cc",
|
|
"symmetric_key.cc",
|
|
],
|
|
hdrs = [
|
|
"aead.h",
|
|
"crypto_export.h",
|
|
"ec_private_key.h",
|
|
"ec_signature_creator.h",
|
|
"ec_signature_creator_impl.h",
|
|
"encryptor.h",
|
|
"hkdf.h",
|
|
"hmac.h",
|
|
"nearby_base.h",
|
|
"openssl_util.h",
|
|
"rsa_private_key.h",
|
|
"secure_hash.h",
|
|
"secure_util.h",
|
|
"sha2.h",
|
|
"signature_verifier.h",
|
|
"symmetric_key.h",
|
|
],
|
|
copts = [
|
|
"-DCRYPTO_IMPLEMENTATION",
|
|
"-Ithird_party",
|
|
],
|
|
deps = [
|
|
"@boringssl//:crypto",
|
|
"@com_google_absl//absl/base:core_headers",
|
|
"@com_google_absl//absl/log",
|
|
"@com_google_absl//absl/log:check",
|
|
"@com_google_absl//absl/status",
|
|
"@com_google_absl//absl/status:statusor",
|
|
"@com_google_absl//absl/strings",
|
|
"@com_google_absl//absl/strings:str_format",
|
|
"@com_google_absl//absl/types:optional",
|
|
"@com_google_absl//absl/types:span",
|
|
],
|
|
)
|
|
|
|
cc_test(
|
|
name = "crypto_cros_unittests",
|
|
size = "large",
|
|
srcs = [
|
|
"aead_unittest.cc",
|
|
"ec_private_key_unittest.cc",
|
|
"ec_signature_creator_unittest.cc",
|
|
"encryptor_unittest.cc",
|
|
"hmac_unittest.cc",
|
|
"rsa_private_key_unittest.cc",
|
|
"secure_hash_unittest.cc",
|
|
"sha2_unittest.cc",
|
|
"signature_verifier_unittest.cc",
|
|
"symmetric_key_unittest.cc",
|
|
],
|
|
copts = [
|
|
"-DUNIT_TEST",
|
|
"-Wno-inconsistent-missing-override",
|
|
"-Wno-non-virtual-dtor",
|
|
"-Ithird_party",
|
|
],
|
|
deps = [
|
|
":crypto_cros",
|
|
"//internal/platform/implementation:types",
|
|
"@com_github_protobuf_matchers//protobuf-matchers",
|
|
"@com_google_absl//absl/strings",
|
|
"@com_google_absl//absl/types:span",
|
|
"@com_google_googletest//:gtest_main",
|
|
],
|
|
)
|