You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2022/10/20 09:21:58 UTC

[GitHub] [commons-net] gremi64 commented on pull request #90: NET-642 using execPROT on FTPSClients with Proxy Settings removes Pro…

gremi64 commented on PR #90:
URL: https://github.com/apache/commons-net/pull/90#issuecomment-1285212799

   Hi, sooo... this PR allow to resolve a bug declared in 2017... but it's not merged because it lacks a unit test to fail without the main changes ?
   Who would create a unit test with a socks proxy to prove it doesn't work as intended ?
   
   I wonder if every fork are made to use this PR as i'll do.
   Thanks @YaniM for the PR :heart: 


-- 
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: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org