You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2022/09/13 12:12:49 UTC
[GitHub] [inlong] healchow opened a new pull request, #5884: [INLONG-5881][Manager] Fix the vulnerability to security attacks for the MySQL JDBC URL
healchow opened a new pull request, #5884:
URL: https://github.com/apache/inlong/pull/5884
### Prepare a Pull Request
- Fixes #5881
### Motivation
Fix the vulnerability to security attacks for the MySQL JDBC URL.
### Modifications
Filter the sensitive params in the MySQL JDBC URL.
### Verifying this change
- [x] This change added tests and can be verified as follows:
-
### Documentation
- Does this pull request introduce a new feature? (no)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] healchow commented on a diff in pull request #5884: [INLONG-5881][Manager] Fix the vulnerability to security attacks for the MySQL JDBC URL
Posted by GitBox <gi...@apache.org>.
healchow commented on code in PR #5884:
URL: https://github.com/apache/inlong/pull/5884#discussion_r970358198
##########
inlong-manager/manager-pojo/pom.xml:
##########
@@ -69,6 +69,18 @@
<groupId>io.swagger</groupId>
<artifactId>swagger-annotations</artifactId>
</dependency>
+ <dependency>
Review Comment:
They were already added to the `licenses/inlong-manager/LICENSE`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] gosonzhang commented on a diff in pull request #5884: [INLONG-5881][Manager] Fix the vulnerability to security attacks for the MySQL JDBC URL
Posted by GitBox <gi...@apache.org>.
gosonzhang commented on code in PR #5884:
URL: https://github.com/apache/inlong/pull/5884#discussion_r970353906
##########
inlong-manager/manager-pojo/pom.xml:
##########
@@ -69,6 +69,18 @@
<groupId>io.swagger</groupId>
<artifactId>swagger-annotations</artifactId>
</dependency>
+ <dependency>
Review Comment:
Need to supplement the LICENSE and Notice of the third-party dependency package (if any)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] dockerzhang merged pull request #5884: [INLONG-5881][Manager] Fix the vulnerability to security attacks for the MySQL JDBC URL
Posted by GitBox <gi...@apache.org>.
dockerzhang merged PR #5884:
URL: https://github.com/apache/inlong/pull/5884
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org