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: asset mgt new endpoints and compile time error for core and config-api #8639

Merged
merged 41 commits into from
Jun 3, 2024
Merged

fix: asset mgt new endpoints and compile time error for core and config-api #8639

merged 41 commits into from
Jun 3, 2024

Conversation

pujavs
Copy link
Contributor

@pujavs pujavs commented Jun 3, 2024 •
edited
Loading

Prepare

Description

  1. Issue#8626: New endpoints for service and asset type
  2. Issue#8627:Fixed compile time issue for core and config-api

Target issue

closes #8626, #8627

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
fix(config-api): user attribute validation error handling
ad74527 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
1ed77f2 
…ns-config-fix
@pujavs
fix(config-api): user attribute validation error handling
1e18060 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix(config-api): user attribute validation error handling
a25977c 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
483ff35 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
aae1be8 
…ns-config-fix
@pujavs
fix: added new IDP attributes
8319446 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix: added new IDP attributes
d608621 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
2c25eb5 
…ns-config-fix
@pujavs
fix(config-api): idp default values set
670d488 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix(config-api): set important fields while configuring IDP to fix au…
9d265d8 
…thentication issue#8384

Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix(config-api): idp metadata default values issue#8384
42a5cd5 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
706eef0 
…ns-config-fix
@pujavs
feat: idp metadata default value and removing filepath in document-store
6a386e6 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
3e4b45c 
…ns-config-fix
@pujavs
feat: idp metadata default value and removing filepath in document-store
f942fe0 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix: assetmgt and idp changes
fd2de6d 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix: assetmgt and idp changes
e7aa37e 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
82ab7f5 
…ns-config-fix
@pujavs
fix(config-api): validation for imp fields of asset mgt
91bb2cd 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
461dffb 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
a54a3ba 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
a51306f 
…ns-config-fix
@pujavs
feat(config-api): pagination for get asset by name, asset filename an…
7868af9 
…d service validation

Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix(config-api): added asset validation, removed redundant fields for TR
c344079 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix(config-api): agama swagger spec changes
899625d 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix(config-api): agama swagger spec changes
aa8b58a 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
fix(config-api): added asset validation, removed redundant fields for TR
c0c8f2b 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
cff488a 
…ns-config-fix
@pujavs
fix(config-api): added asset validation, removed redundant fields for TR
7ef4699 
Signed-off-by: pujavs <pujas.works@gmail.com>
pujavs added 11 commits May 24, 2024 22:49
@pujavs
fix(config-api): added asset validation, removed redundant fields for TR
755c38a 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
bb308e4 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
a1c4f4c 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
73884cc 
…ns-config-fix
@pujavs
feat(config-api): user mgt error handling
cae338b 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
3b01e5e 
…ns-config-fix
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
1630c6b 
…ns-config-fix
@pujavs
feat(config-api): user mgt error handling
4c96b9d 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
feat(config-api): user mgt error handling
7d1d979 
Signed-off-by: pujavs <pujas.works@gmail.com>
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
33f3b3b 
…ns-config-fix
@pujavs
feat(config-api): asset mgt service module changes
8ab7a09 
Signed-off-by: pujavs <pujas.works@gmail.com>
@mo-auto mo-auto added comp-jans-config-api Component affected by issue or PR comp-jans-core 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 Jun 3, 2024
@pujavs pujavs mentioned this pull request Jun 3, 2024
Closed
@pujavs pujavs changed the title fix compile time error for core and config-api fix: asset mgt new endpoints and compile time error for core and config-api Jun 3, 2024
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jun 3, 2024

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

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jun 3, 2024

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

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jun 3, 2024

Quality Gate Passed Quality Gate passed for 'jans-config-api-parent'

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@yuriyz yuriyz enabled auto-merge (squash) June 3, 2024 14:59
@yuriyz yuriyz merged commit 1c6ecfd into main Jun 3, 2024
5 checks passed
@yuriyz yuriyz deleted the jans-config-fix branch June 3, 2024 15:29
@pujavs pujavs mentioned this pull request Jun 3, 2024
Closed
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

@devrimyatar devrimyatar devrimyatar approved these changes

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

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

Assignees
No one assigned
Labels
comp-jans-config-api Component affected by issue or PR comp-jans-core 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.

feat(config-api): asset mgt endpoint to return service and standard file type supported
4 participants
@pujavs @yuriyz @devrimyatar @mo-auto