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

Configure sensitive fields that should be redacted #3687

Open
3 of 5 tasks
JohnDuncanScott opened this issue Dec 29, 2023 · 1 comment
Open
3 of 5 tasks

Configure sensitive fields that should be redacted #3687

JohnDuncanScott opened this issue Dec 29, 2023 · 1 comment
Labels
feat New feature or request.

Comments

@JohnDuncanScott
Copy link

Preflight checklist

Ory Network Project

No response

Describe your problem

Hydra already redacts some fields for you, such as "cookie". There are other fields that should (according to our company guidelines) also be redacted, such as:
headers:
cf-connecting-ip
forwarded
x-forwarded-for

which contain IP address, which can be deemed as sensitive.

I have searched the Hydra docs and the issues in this repo and it does not appear that it's possible to choose what fields get redacted. In this particular case, it would be great to redact more fields by default (rather than the existing config option of showing the sensitive data).

Describe your ideal solution

Configuration option for Hydra to list other fields that should be redacted in addition to the default ones. This list would be merged with the internal one that contains "cookies", "query", etc..

Workarounds or alternatives

If there is a way of doing this that's undocumented in Hydra, that would be great to know :). This data could be scrubbed elsewhere potentially, like in DataDog for example, but since you're already doing redaction and just need to expose some config to add to it, I think it's a reasonable request?

Version

oryd/hydra:v1.11.10

Additional Context

No response

@JohnDuncanScott JohnDuncanScott added the feat New feature or request. label Dec 29, 2023
@alnr
Copy link
Contributor

alnr commented Mar 28, 2024

Same issue: ory/oathkeeper#1081

Needs a fix in ory/x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request.
Projects
None yet
Development

No branches or pull requests

2 participants