Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Immutables redactions may be unsafe (previously forced do-not-log) #2277

Merged
merged 2 commits into from
May 16, 2022

Conversation

carterkozak
Copy link
Contributor

We still assume do-not-log by default, however type and annotation
information may be used to suggest unsafe.

this addresses an edge case that I've seen come up from time to time.

==COMMIT_MSG==
Immutables redactions may be unsafe (previously forced do-not-log)
==COMMIT_MSG==

We still assume do-not-log by default, however type and annotation
information may be used to suggest unsafe.
@changelog-app
Copy link

changelog-app bot commented May 16, 2022

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Immutables redactions may be unsafe (previously forced do-not-log)

Check the box to generate changelog(s)

  • Generate changelog entry

@bulldozer-bot bulldozer-bot bot merged commit 7394f8e into develop May 16, 2022
@bulldozer-bot bulldozer-bot bot deleted the ckozak/redactions_may_be_unsafe branch May 16, 2022 15:40
@svc-autorelease
Copy link
Collaborator

Released 4.137.0

This was referenced May 16, 2022
This was referenced May 16, 2022
bulldozer-bot bot pushed a commit to palantir/witchcraft-api that referenced this pull request May 16, 2022
###### _excavator_ is a bot for automating changes across repositories.

Changes produced by the roomba/latest-baseline-oss check.

# Release Notes
## 4.137.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Improvement | Immutables redactions may be unsafe (previously forced do-not-log) | palantir/gradle-baseline#2277 |


## 4.138.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Fix | Removed built in support for eclipse annotation processor configurations - please use [gradle-processors](https://github.com/palantir/gradle-processors) instead if using eclipse. | palantir/gradle-baseline#2279 |


## 4.139.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Fix | Rerelease due to failed publish | palantir/gradle-baseline#2280 |



To enable or disable this check, please contact the maintainers of Excavator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants