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.

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

feat(jans-auth-server): remove dcrSkipSignatureValidation configuration property #1623

Closed
yuriyz opened this issue Jun 28, 2022 · 1 comment
Closed

feat(jans-auth-server): remove dcrSkipSignatureValidation configuration property #1623

yuriyz opened this issue Jun 28, 2022 · 1 comment
Assignees
@yuriyz
Labels
comp-jans-auth-server Component affected by issue or PR kind-feature Issue or PR is a new feature request

Comments

@yuriyz
Copy link
Contributor

yuriyz commented Jun 28, 2022

Description

We have dcrSignatureValidationEnabled configuration property which enables/disabled dcr validation. We don't need second one for same confuration, we should remove "dcrSkipSignatureValidation" configuration property
@yuriyz yuriyz added comp-jans-auth-server Component affected by issue or PR kind-feature Issue or PR is a new feature request labels Jun 28, 2022
@yuriyz yuriyz self-assigned this Jun 28, 2022
yuriyz added a commit that referenced this issue Jun 28, 2022
@yuriyz
feat(jans-auth-server): removed dcrSkipSignatureValidation configurat…
6550247 
…ion property #1623

#1623
@yuriyzz yuriyzz mentioned this issue Jun 28, 2022
Merged
yuriyz added a commit that referenced this issue Jun 28, 2022
@yuriyz
Merge pull request #1624 from JanssenProject/jans-auth-server-1623
b3b3d22 
feat(jans-auth-server): removed dcrSkipSignatureValidation configuration property #1623
@yuriyz
Copy link
Contributor Author

yuriyz commented Jun 28, 2022

Done

@yuriyz yuriyz closed this as completed Jun 28, 2022
@mo-auto mo-auto mentioned this issue Jul 5, 2022
Merged
@mo-auto2 mo-auto2 mentioned this issue Jul 6, 2022
Merged
This was referenced Jul 6, 2022
Merged
Merged
@mo-auto mo-auto mentioned this issue Jul 6, 2022
Merged
This was referenced Aug 30, 2022
Merged
Merged
@mo-auto2 mo-auto2 mentioned this issue Aug 30, 2022
Merged
This was referenced Aug 30, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuriyz yuriyz

Labels
comp-jans-auth-server Component affected by issue or PR kind-feature Issue or PR is a new feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yuriyz