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

fix(jans-cli-tui): don't include client secret in summary #7161

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

devrimyatar
Copy link
Contributor

closes #7160

Signed-off-by: Mustafa Baser <mbaser@mail.com>
@devrimyatar devrimyatar added kind-bug Issue or PR is a bug in existing functionality comp-jans-cli-tui Component affected by issue or PR labels Dec 20, 2023
Copy link

sonarcloud bot commented Dec 20, 2023

Quality Gate Passed Quality Gate passed for 'jans-cli'

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@yuriyz yuriyz merged commit 798a4d7 into main Dec 20, 2023
3 checks passed
@yuriyz yuriyz deleted the jans-cli-tui-client-secret branch December 20, 2023 11:09
yurem pushed a commit that referenced this pull request Dec 21, 2023
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Signed-off-by: Yuriy Movchan <Yuriy.Movchan@gmail.com>
devrimyatar added a commit that referenced this pull request Dec 30, 2023
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-cli-tui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix(jans-cli-tui): don't save client secret in summary
3 participants