-
Notifications
You must be signed in to change notification settings - Fork 100
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
(chores): fix security vulnerabilities #1482
(chores): fix security vulnerabilities #1482
Conversation
TOMCAT deployment fails due to following error:
I tried adding the new Attached the file with complete error log while deploying the application in tomcat: Modified the |
8c8a5da
to
5904aa3
Compare
The above issue has been fixed. But, I can see the following error in deployment logs, where as sw360 application is working fine and I don't see any more issues with tomcat deployment of web application.
|
7659cfe
to
fbbf933
Compare
This PR include following changes:
Task to be done in future:
TESTINGFor Testing the PR, You have to place the |
fbbf933
to
7fd6302
Compare
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.
Reviewed the PR.
There seems to be some issue with
- Health api Response structure
- Hal Browser
- Url changed
- Asking for Password in stage
...urce-server/src/test/java/org/eclipse/sw360/rest/resourceserver/restdocs/HealthSpecTest.java
Outdated
Show resolved
Hide resolved
...urce-server/src/test/java/org/eclipse/sw360/rest/resourceserver/restdocs/HealthSpecTest.java
Outdated
Show resolved
Hide resolved
...urce-server/src/test/java/org/eclipse/sw360/rest/resourceserver/restdocs/HealthSpecTest.java
Outdated
Show resolved
Hide resolved
7fd6302
to
c9be58e
Compare
Health API (
HAL browser URL is not something managed by us, and the URL changes is due to change in spring-boot version, from version 2.2 onwards I need to look into password issue. Thanks. |
c9be58e
to
01710be
Compare
Issue has been fixed. |
01710be
to
e4ea1e5
Compare
Signed-off-by: Abdul Kapti <[email protected]>
e4ea1e5
to
2502b58
Compare
Issue: #1481
Suggest Reviewer
Current Issue:
I have fixed all the test cases, but the liferay fails while deployment. we need to fix the liferay deployment issues.
How To Test?
Please test all the functionalities in UI and REST API is working fine.
Checklist
Must:
Signed-off-by: Abdul Kapti [email protected]