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: documentstore changes for asset mgt and idp changes for new fields #8530

Merged
merged 18 commits into from
May 18, 2024

Conversation

pujavs
Copy link
Contributor

@pujavs pujavs commented May 17, 2024

Prepare


Description

  1. Document-store changes for reverting jansFilePath field changes. And for jansRevision dataType mismatch issue.
  2. Asset-mgt changes for removing reference of jansFilePath and for jansRevision field handling.
  3. SAML idp changes for adding new fields.

Target issue

closes #8384 #8529

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)

Signed-off-by: pujavs <pujas.works@gmail.com>
Signed-off-by: pujavs <pujas.works@gmail.com>
Signed-off-by: pujavs <pujas.works@gmail.com>
Signed-off-by: pujavs <pujas.works@gmail.com>
Signed-off-by: pujavs <pujas.works@gmail.com>
Signed-off-by: pujavs <pujas.works@gmail.com>
…thentication issue#8384

Signed-off-by: pujavs <pujas.works@gmail.com>
Signed-off-by: pujavs <pujas.works@gmail.com>
Signed-off-by: pujavs <pujas.works@gmail.com>
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 May 17, 2024
@yuriyz yuriyz enabled auto-merge (squash) May 17, 2024 16:31
Copy link

sonarcloud bot commented May 17, 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

Copy link

sonarcloud bot commented May 17, 2024

Copy link

sonarcloud bot commented May 17, 2024

@yuriyz yuriyz merged commit a1cd331 into main May 18, 2024
5 checks passed
@yuriyz yuriyz deleted the jans-config-fix branch May 18, 2024 08:37
yuriyz pushed a commit that referenced this pull request Nov 7, 2024
…for new fields (#8530)

* fix(config-api): user attribute validation error handling

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix(config-api): user attribute validation error handling

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix(config-api): user attribute validation error handling

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix: added new IDP attributes

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix: added new IDP attributes

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix(config-api): idp default values set

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix(config-api): idp metadata default values issue#8384

Signed-off-by: pujavs <pujas.works@gmail.com>

* feat: idp metadata default value and removing filepath in document-store

Signed-off-by: pujavs <pujas.works@gmail.com>

* feat: idp metadata default value and removing filepath in document-store

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix: assetmgt and idp changes

Signed-off-by: pujavs <pujas.works@gmail.com>

* fix: assetmgt and idp changes

Signed-off-by: pujavs <pujas.works@gmail.com>

---------

Signed-off-by: pujavs <pujas.works@gmail.com>
Former-commit-id: a1cd331
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

fix(config-api): set important fields while configuring IDP to fix authentication issue
4 participants