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: Use highest level script in case ACR script is not found #1070

Merged

Conversation

Milton-Ch
Copy link
Contributor

Description

  1. Getting highest level script in case ACR script is not found.
  2. Added FF in config to enable/disable this behavior.

@sonarcloud
Copy link

sonarcloud bot commented Mar 18, 2022

[jans-cli] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sonarcloud
Copy link

sonarcloud bot commented Mar 18, 2022

[jans-config-api-parent] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sonarcloud
Copy link

sonarcloud bot commented Mar 18, 2022

[jans-linux-setup] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sonarcloud
Copy link

sonarcloud bot commented Mar 18, 2022

[Jans authentication server parent] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Milton-Ch Milton-Ch force-pushed the fix/use-highest-level-script-in-case-acr-script-not-found branch from fc0ae0f to 9d1d0b5 Compare March 22, 2022 12:41
@Milton-Ch
Copy link
Contributor Author

Successful Jenkins execution: https://jenkins.jans.io/jenkins/job/jans-auth-server/5924/

@Milton-Ch Milton-Ch marked this pull request as ready for review March 22, 2022 13:20
yuriyz
yuriyz previously approved these changes Mar 22, 2022
@Milton-Ch Milton-Ch enabled auto-merge (squash) March 22, 2022 17:41
moabu
moabu previously approved these changes Mar 22, 2022
iromli
iromli previously approved these changes Mar 22, 2022
devrimyatar
devrimyatar previously approved these changes Mar 23, 2022
@Milton-Ch Milton-Ch dismissed stale reviews from devrimyatar, iromli, moabu, and yuriyz via b90f1e1 March 23, 2022 13:09
@Milton-Ch Milton-Ch force-pushed the fix/use-highest-level-script-in-case-acr-script-not-found branch from 9d1d0b5 to b90f1e1 Compare March 23, 2022 13:09
@moabu moabu self-requested a review March 23, 2022 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants