Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

feat: remove not needed encryption of secrets#1123

Merged
yrobla merged 9 commits into
mainstacklok/codegate:mainfrom
issue-929stacklok/codegate:issue-929Copy head branch name to clipboard
Mar 4, 2025
Merged

feat: remove not needed encryption of secrets#1123
yrobla merged 9 commits into
mainstacklok/codegate:mainfrom
issue-929stacklok/codegate:issue-929Copy head branch name to clipboard

Conversation

@yrobla

@yrobla yrobla commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

Instead use an uuid generator as we do for pii, and reuse same session store mechanism

Closes: #929

@yrobla yrobla marked this pull request as draft February 20, 2025 10:49
@yrobla yrobla marked this pull request as ready for review February 21, 2025 10:41
@yrobla yrobla requested review from aponcedeleonch, jhrozek and lukehinds and removed request for aponcedeleonch and lukehinds February 21, 2025 10:43

@blkt blkt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a few comments.

Comment thread src/codegate/pipeline/base.py
Comment thread src/codegate/pipeline/sensitive_data/manager.py Outdated
Comment thread src/codegate/pipeline/pii/pii.py Outdated
Comment thread src/codegate/pipeline/pii/pii.py Outdated
Comment thread src/codegate/pipeline/pii/pii.py Outdated
Comment thread src/codegate/pipeline/sensitive_data/manager.py Outdated
Comment thread src/codegate/pipeline/sensitive_data/manager.py Outdated
@yrobla yrobla requested review from blkt and jhrozek February 24, 2025 08:53
@yrobla yrobla force-pushed the issue-929 branch 6 times, most recently from da2f6a8 to 0861a59 Compare February 24, 2025 09:32
Comment thread src/codegate/pipeline/pii/pii.py Outdated
@yrobla yrobla requested a review from jhrozek March 4, 2025 13:01
@yrobla yrobla merged commit 58132ba into main Mar 4, 2025
@yrobla yrobla deleted the issue-929 branch March 4, 2025 13:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor redaction mechanism to a single interface

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.