-
Notifications
You must be signed in to change notification settings - Fork 189
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
Ftps proxy error #154
base: master
Are you sure you want to change the base?
Ftps proxy error #154
Conversation
@garydgregory Running this test case requires the use of Docker Compose. Are the Commons Team amenable to have test cases adding to your unit tests or integration tests that rely on Docker Compose? |
I'm OK with it. |
Hm, I can't get the reproducer to work.
and running the using test cannot connect:
|
I tried the test myself on my mac and I'm getting an issue with the SSL handshake due to a cert having expired.
|
I got the test to run on my mac by regenerating the pem file.
key.pem and cert.pem are then manually merged to created pure-ftp.pem The tests failed as expected with this change and when I applied potential solution b in https://github.com/apache/commons-net/pull/154/files#diff-cbb58f257b18c23645163433a3a2f989d76df71e55c5a2ac77bba86ed4540215 |
Yeah, I'm not even getting that far, the test can't seem to connect. Maybe the port is misconfigured? |
Do you have Squid running? |
I followed the Homebrew instructions... |
But what's confusing is that it's not clear (to me) if you need the Docker part of the setup on macOS. If I could get clear macOS instructions that would help ... |
As requested in https://issues.apache.org/jira/browse/NET-718
FYI @garydgregory