-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature/OIDC integration2 #350
Conversation
…ng root path to opeanpi swagger ui, removing redundant default index.gsp
…ng back previously removed dependenies
Required ansible configs have been added under feature branches. (To merge after this app is ready to deploy with OIDC) |
gradle.properties
Outdated
grailsWrapperVersion=1.0.0 | ||
gormVersion=6.0.12.RELEASE | ||
gradleWrapperVersion=3.4.1 | ||
grailsVersion=4.0.13 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please upgrade to 4.1.1 or to any secured version
More info: https://grails.org/blog/2022-07-18-rce-vulnerability.html
@@ -58,7 +58,7 @@ class UrlMappings { | |||
|
|||
"/logout/logout" (controller: 'misc', action: 'logout') | |||
|
|||
"/"(view: "/index") | |||
"/"(redirect: "/openapi") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This tires to maintain the behaviour of the current bie-index's index /
page which displays the swagger UI under the /index
path. New changes replace the old swagger iplementation with openapi-1.0 for api spec definition and this change displayes the /opeanapi
page for the /
since old index page no longer exists.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please check comments?
build.gradle
Outdated
@@ -102,6 +102,8 @@ dependencies { | |||
testCompile "org.seleniumhq.selenium:selenium-support:3.14.0" | |||
testRuntime "org.seleniumhq.selenium:selenium-chrome-driver:3.14.0" | |||
testRuntime "org.seleniumhq.selenium:selenium-firefox-driver:3.14.0" | |||
// https://mavenlibs.com/maven/dependency/org.springframework/spring-messaging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we remove this if it is necessary?
20d79f5
to
91d4405
Compare
Grails4 upgrade and and OIDC in Integration