Skip to content
This repository was archived by the owner on Dec 20, 2023. It is now read-only.

Commit c47a9cc

Browse files
authored
Merge pull request #52 from aiw-google/aiw-google-clearer-warnings
Clearer 'security disabled' warnings for Dev builds
2 parents da3e76f + 75b14cd commit c47a9cc

File tree

1 file changed

+16
-4
lines changed

1 file changed

+16
-4
lines changed

src/lib/core/WeaveFabricState.cpp

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,23 @@ using namespace nl::Weave::Profiles::FabricProvisioning;
6565
using namespace nl::Weave::Profiles::Security::AppKeys;
6666

6767
#if WEAVE_CONFIG_SECURITY_TEST_MODE
68+
#pragma message "\n \
69+
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!\n \
70+
!!!! WARNING - SECURITY_TEST_MODE IS ENABLED !!!!\n \
71+
!!!! BASIC WEAVE SECURITY / ENCRYPTION IS CRIPPLED !!!!\n \
72+
!!!! DEVELOPMENT ONLY -- DO NOT SHIP !!!!\n \
73+
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!\n \
74+
"
75+
#endif
6876
69-
#define EXCLS "\n!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"
70-
#pragma message (EXCLS "\n!!!! WARNING - WEAVE SECURITY TEST MODE ENABLED !!!!" EXCLS)
71-
#undef EXCLS
72-
77+
#if !WEAVE_CONFIG_REQUIRE_AUTH
78+
#pragma message "\n \
79+
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!\n \
80+
!!!! WARNING - REQUIRE_AUTH IS DISABLED !!!!\n \
81+
!!!! CLIENT AUTHENTICATION IS NOT REQUIRED !!!!\n \
82+
!!!! DEVELOPMENT ONLY -- DO NOT SHIP !!!!\n \
83+
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!\n \
84+
"
7385
#endif
7486
7587
#ifndef nlDEFINE_ALIGNED_VAR

0 commit comments

Comments
 (0)