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

fix(kc-saml-plugin): exception handling and error handling #7351

Merged
merged 45 commits into from
Jan 12, 2024
Merged

fix(kc-saml-plugin): exception handling and error handling #7351

merged 45 commits into from
Jan 12, 2024

Conversation

pujavs
Copy link
Contributor

@pujavs pujavs commented Jan 12, 2024

Prepare

Description

Target issue

closes #7316

Implementation Details

Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

pujavs added 30 commits December 27, 2023 23:16
@pujavs
fix(kc-saml-plugin): remove kc lib dependency - wip
564241f 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
7e5db38 
…ns-config-fix
@pujavs
fix(kc-saml-plugin): removing dependency of kc lib wip
c64cc58 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(kc-saml-plugin): removing dependency of kc lib wip
2c3e11e 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(config-api) keycloak lib conflict issue
4e69482 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(config-api) keycloak lib conflict issue
2e0e53d 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(config-api) keycloak lib conflict issue
3640052 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(config-api) keycloak lib conflict issue
d20392b 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(config-api) keycloak lib conflict issue
6318fe4 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
9bd5ae5 
…ns-config-fix
@pujavs
fix(kc-config-api): removing kc lib dependency - wip
5de05ca 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(kc-config-api): removing kc lib dependency - wip
9a0e889 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(kc-config-api): removing kc lib dependency - wip
30b2d4c 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
d54c7f4 
…ns-config-fix
@pujavs
feat(kc-config-api): removed kc lib dependency
cf1880b 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-config-api): removed kc lib dependency
185b3c1 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-config-api): removed kc lib dependency
06ad38c 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-config-api): removed kc lib dependency
c7e3a9a 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-config-api): sync with main
bc186a3 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-config-api): sync with main
ab4799d 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-saml-plugin): sonar code issue fixed
e63ef9e 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-saml-plugin): saml plugin config changes
dd0a9e6 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-saml-plugin): spec changes
ba617e3 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-saml-plugin): saml config changes
cef74d9 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-saml-plugin): saml config changes
ab703d2 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
1d4f4e4 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
846e96f 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
02f0191 
…ns-config-fix
@pujavs
fix(kc-saml-plugin): put endpoint fix
3e14209 
Signed-off-by: pujavs <[email protected]>
@pujavs
fix(kc-saml-plugin): put endpoint fix
717c990 
Signed-off-by: pujavs <[email protected]>
pujavs added 15 commits January 8, 2024 12:44
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
b46012e 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
012a163 
…ns-config-fix
@pujavs
feat(config-api): version data changes issue#7284
d26408b 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
332dbad 
…ns-config-fix
@pujavs
feat(config-api): exception handling
34639b9 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(config-api): exception handling
cf3100c 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-saml-plugin): error handling and setting default realm
49a774a 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
2d028dd 
…ns-config-fix
@pujavs
feat(kc-saml-plugin): error handling and setting default realm
1566435 
Signed-off-by: pujavs <[email protected]>
@pujavs
feat(kc-saml-plugin): error handling and setting default realm
cacb584 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
2b93ce3 
…ns-config-fix
@pujavs
feat(kc-saml-plugin): error handling and setting default realm
fa6ee2c 
Signed-off-by: pujavs <[email protected]>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
97784c8 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
422f32b 
…ns-config-fix
@pujavs
feat(kc-saml-plugin): error handling
ac73413 
Signed-off-by: pujavs <[email protected]>
@mo-auto mo-auto added comp-jans-config-api Component affected by issue or PR comp-jans-linux-setup Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Jan 12, 2024
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed for 'jans-linux-setup'

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@yurem yurem merged commit afe15ff into main Jan 12, 2024
2 checks passed
@yurem yurem deleted the jans-config-fix branch January 12, 2024 16:16
@mo-auto3 mo-auto3 mentioned this pull request Jan 22, 2024
Merged
yuriyz pushed a commit that referenced this pull request Nov 7, 2024
@pujavs
fix(kc-saml-plugin): exception handling and error handling (#7351)
fd16d84 
* fix(kc-saml-plugin): remove kc lib dependency - wip

Signed-off-by: pujavs <[email protected]>

* fix(kc-saml-plugin): removing dependency of kc lib wip

Signed-off-by: pujavs <[email protected]>

* fix(kc-saml-plugin): removing dependency of kc lib wip

Signed-off-by: pujavs <[email protected]>

* fix(config-api) keycloak lib conflict issue

Signed-off-by: pujavs <[email protected]>

* fix(config-api) keycloak lib conflict issue

Signed-off-by: pujavs <[email protected]>

* fix(config-api) keycloak lib conflict issue

Signed-off-by: pujavs <[email protected]>

* fix(config-api) keycloak lib conflict issue

Signed-off-by: pujavs <[email protected]>

* fix(config-api) keycloak lib conflict issue

Signed-off-by: pujavs <[email protected]>

* fix(kc-config-api): removing kc lib dependency - wip

Signed-off-by: pujavs <[email protected]>

* fix(kc-config-api): removing kc lib dependency - wip

Signed-off-by: pujavs <[email protected]>

* fix(kc-config-api): removing kc lib dependency - wip

Signed-off-by: pujavs <[email protected]>

* feat(kc-config-api): removed kc lib dependency

Signed-off-by: pujavs <[email protected]>

* feat(kc-config-api): removed kc lib dependency

Signed-off-by: pujavs <[email protected]>

* feat(kc-config-api): removed kc lib dependency

Signed-off-by: pujavs <[email protected]>

* feat(kc-config-api): removed kc lib dependency

Signed-off-by: pujavs <[email protected]>

* feat(kc-config-api): sync with main

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): sonar code issue fixed

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): saml plugin config changes

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): saml config changes

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): saml config changes

Signed-off-by: pujavs <[email protected]>

* fix(kc-saml-plugin): put endpoint fix

Signed-off-by: pujavs <[email protected]>

* fix(kc-saml-plugin): put endpoint fix

Signed-off-by: pujavs <[email protected]>

* feat(config-api): version data changes issue#7284

Signed-off-by: pujavs <[email protected]>

* feat(config-api): exception handling

Signed-off-by: pujavs <[email protected]>

* feat(config-api): exception handling

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): error handling and setting default realm

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): error handling and setting default realm

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): error handling and setting default realm

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): error handling and setting default realm

Signed-off-by: pujavs <[email protected]>

* feat(kc-saml-plugin): error handling

Signed-off-by: pujavs <[email protected]>

---------

Signed-off-by: pujavs <[email protected]>
Former-commit-id: afe15ff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yuriyz yuriyz yuriyz approved these changes

@yurem yurem yurem approved these changes

@devrimyatar devrimyatar Awaiting requested review from devrimyatar devrimyatar is a code owner

Assignees
No one assigned
Labels
comp-jans-config-api Component affected by issue or PR comp-jans-linux-setup Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix(kc-saml-plugin): exception handling and setting default realm
4 participants
@pujavs @yuriyz @yurem @mo-auto