Skip to content

Conversation

@ardaguclu
Copy link
Member

This PR prohibits any operation with regards to Secret resource.

@manusa
Copy link
Member

manusa commented Jun 18, 2025

This should be configurable.

Somehow related to #129
Might require #131

@manusa
Copy link
Member

manusa commented Jun 18, 2025

Original idea was to provide an allolist and denylist #132

Which also relates to providing configuration through toml file

@ardaguclu
Copy link
Member Author

Thanks for the pointers. It is better to have general approach as you mentioned. Closing...

@ardaguclu ardaguclu closed this Jun 18, 2025
@ardaguclu ardaguclu deleted the prohibit-secret branch June 18, 2025 11:15
@manusa
Copy link
Member

manusa commented Jun 18, 2025

Thanks for the pointers. It is better to have general approach as you mentioned. Closing...

np, I think it should be better to see how to tackle #131 first to be able to easily provide a deny list (for example).
Implementation after that should then be extremely easy, it could even be done with a generic interceptor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants