You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "abhioncbr (via GitHub)" <gi...@apache.org> on 2023/07/16 16:26:46 UTC
[GitHub] [pinot] abhioncbr opened a new issue, #11118: Delete the usage of the old commons-httpclient dependency
abhioncbr opened a new issue, #11118:
URL: https://github.com/apache/pinot/issues/11118
Currently, pinot has the dependency both on [commons-httpclient](https://github.com/apache/pinot/blob/master/pom.xml#L523C18-L523C36) and [httpcomponents](https://github.com/apache/pinot/blob/master/pom.xml#L302).
I think using the two versions of the same library doesn't make much sense. We should delete the use of the old version, i.e. `commons-httpclient`. Also, the removal of this dependency helps us get rid of some of the vulnerabilities because of the library.
- `commons-httpclient` is getting used in a handful of classes, and to remove the dependency, we need to make the changes in the classes using it.
I can work on it if this looks good. Thanks
cc: @Jackie-Jiang
labels: dependency
--
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@pinot.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] Jackie-Jiang commented on issue #11118: Delete the usage of the old commons-httpclient dependency
Posted by "Jackie-Jiang (via GitHub)" <gi...@apache.org>.
Jackie-Jiang commented on issue #11118:
URL: https://github.com/apache/pinot/issues/11118#issuecomment-1644370534
I'm +1 to this
--
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@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] abhioncbr commented on issue #11118: Delete the usage of the old commons-httpclient dependency
Posted by "abhioncbr (via GitHub)" <gi...@apache.org>.
abhioncbr commented on issue #11118:
URL: https://github.com/apache/pinot/issues/11118#issuecomment-1675866447
Done.
--
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@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] xiangfu0 commented on issue #11118: Delete the usage of the old commons-httpclient dependency
Posted by "xiangfu0 (via GitHub)" <gi...@apache.org>.
xiangfu0 commented on issue #11118:
URL: https://github.com/apache/pinot/issues/11118#issuecomment-1670569350
+1
--
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@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] abhioncbr closed issue #11118: Delete the usage of the old commons-httpclient dependency
Posted by "abhioncbr (via GitHub)" <gi...@apache.org>.
abhioncbr closed issue #11118: Delete the usage of the old commons-httpclient dependency
URL: https://github.com/apache/pinot/issues/11118
--
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@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org