Skip to content

Commit e934a69

Browse files
author
Darin Howard
committed
RT-44 - updating masking to be disabled by default
1 parent 6bb5c1f commit e934a69

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,9 @@ stackify.environment=YOUR_ENVIRONMENT
4646

4747
The Stackify logger has built-in data masking for credit cards and social security number values.
4848

49-
**Disable Masking:**
49+
**Enable Masking:**
5050

51-
Add `stackify.log.mask.enabled=false` to `stackify-api.properties`.
51+
Add `stackify.log.mask.enabled=true` to `stackify-api.properties`.
5252

5353
**Customize Masking:**
5454

src/main/java/com/stackify/api/common/mask/MaskerConfiguration.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,11 @@ public static Masker fromProperties(String propertiesFilePath) {
5050

5151
Map<String, String> map = PropertyUtil.read(propertiesFilePath);
5252

53-
if (map.containsKey("stackify.log.mask.enabled")) {
54-
if (!Boolean.parseBoolean(map.get("stackify.log.mask.enabled"))) {
55-
masker.clearMasks();
56-
return masker;
57-
}
53+
// masker defaults to disabled
54+
if (!map.containsKey("stackify.log.mask.enabled") ||
55+
!Boolean.parseBoolean(map.get("stackify.log.mask.enabled"))) {
56+
masker.clearMasks();
57+
return masker;
5858
}
5959

6060
// set masks

0 commit comments

Comments
 (0)