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

Secrets are decoded upon describe #2461

Merged

Conversation

placintaalexandru
Copy link
Contributor

@placintaalexandru placintaalexandru commented Jan 10, 2024

Fixes #373

Copy link
Owner

@derailed derailed left a comment

Choose a reason for hiding this comment

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

@placintaalexandru Thank you Alexandru for taking this on!!
What you have here would be just fine with some TLC.
But wondering if this could be an opportunity to refactor and improve so we can build on in the future?

internal/view/secret.go Outdated Show resolved Hide resolved
internal/view/secret.go Outdated Show resolved Hide resolved
internal/view/secret.go Outdated Show resolved Hide resolved
Copy link
Owner

@derailed derailed left a comment

Choose a reason for hiding this comment

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

@placintaalexandru Thanks for these updates! Getting closer...

internal/dao/secret.go Outdated Show resolved Hide resolved
internal/dao/secret.go Show resolved Hide resolved
internal/dao/secret.go Outdated Show resolved Hide resolved
internal/dao/secret.go Show resolved Hide resolved
internal/dao/secret.go Show resolved Hide resolved
internal/render/secret.go Outdated Show resolved Hide resolved
@derailed derailed added enhancement New feature or request needs-tlc Pr needs additional updates labels Jan 18, 2024
@placintaalexandru placintaalexandru force-pushed the feat/Secrets-are-decoded branch 2 times, most recently from b3140d1 to df69476 Compare January 18, 2024 16:25
Copy link
Owner

@derailed derailed left a comment

Choose a reason for hiding this comment

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

@placintaalexandru Thank you for the updates Alexandru!

internal/dao/secret.go Outdated Show resolved Hide resolved
internal/dao/secret.go Outdated Show resolved Hide resolved
Copy link
Owner

@derailed derailed left a comment

Choose a reason for hiding this comment

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

@placintaalexandru Thank you for the updates Alexandru!

@derailed derailed merged commit 1a2ee10 into derailed:master Jan 19, 2024
2 of 3 checks passed
@placintaalexandru placintaalexandru deleted the feat/Secrets-are-decoded branch January 19, 2024 08:57
thejoeejoee pushed a commit to thejoeejoee/k9s that referenced this pull request Feb 23, 2024
placintaalexandru added a commit to placintaalexandru/k9s that referenced this pull request Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-tlc Pr needs additional updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Secret decoded content is not shown
2 participants