Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Hide sensitive information with SensitiveParameter attribute #16964

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.

Sign up for GitHub

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

Jump to bottom
Closed
fabpot opened this issue Jul 11, 2022 · 2 comments
Closed

Hide sensitive information with SensitiveParameter attribute #16964

fabpot opened this issue Jul 11, 2022 · 2 comments
Labels
hasPR A Pull Request has already been submitted for this issue.
Milestone
6.2

Comments

@fabpot
Copy link
Member

fabpot commented Jul 11, 2022

Q A
Feature PR symfony/symfony#46183
PR author(s) @GromNaN
Merged in 6.2
@alexandre-daubois
Copy link
Member

I think this could be added as a .. note: somewhere in the Security documentation just to inform developers of this from PHP 8.2. But I can't exactly decide where to add this, in the Security documentation?

@xabbuh xabbuh added this to the 6.2 milestone Jul 12, 2022
@javiereguiluz javiereguiluz mentioned this issue Jul 18, 2022
Merged
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Jul 18, 2022
javiereguiluz added a commit that referenced this issue Jul 19, 2022
@javiereguiluz
minor #16997 [Security] Mention the SensitiveParameter attribute (jav…
e446184 
…iereguiluz)

This PR was squashed before being merged into the 6.2 branch.

Discussion
----------

[Security] Mention the SensitiveParameter attribute

Fixes #16964.

When Symfony 7.0 is published, instead of removing this `versionadded` directive, we can turn it into a `tip` directive.

Commits
-------

facd25d [Security] Mention the SensitiveParameter attribute
@javiereguiluz
Copy link
Member

Fixed in #16997.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
6.2
Development

No branches or pull requests
4 participants
@fabpot @javiereguiluz @xabbuh @alexandre-daubois