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

Add sensitive support #828

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

bdeferme
Copy link

Pull Request (PR) description

Adds sensitive support for passwords

This Pull Request (PR) fixes the following issues

Fixes #440

@bdeferme
Copy link
Author

bdeferme commented Sep 7, 2022

Rebased on newer master

@teluq-pbrideau
Copy link
Contributor

I would also be interrested in this feature, can I help in any way?
@bdeferme Could you rebase from master to fix the test about web.pp and make it go forward?

@bdeferme
Copy link
Author

I rebased but some tests (some related to my code, some not) are failing. I can't find the time atm too look at this. If you could @teluq-pbrideau go ahead. Otherwise I will maybe find some time next year :-)

@teluq-pbrideau
Copy link
Contributor

Thanks @bdeferme, as you see, i’ve duplicated to a new PR, where I will be able to work easily.
I’m not sure it is the best way to collaborate, feel free if you think I should do otherwise

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.

Manage resources leaks API password in resource types
3 participants