enasequence / webin-cli

Webin command line submission program.
Apache License 2.0
30 stars 5 forks source link

Issue running behind proxy #9

Closed seretol closed 5 years ago

seretol commented 5 years ago

webin cli 1.6.0 doesn't seem to work when behind a proxy even when specifying the https.proxyHost and https.proxyPort properties. I ran the following: java -Dhttps.proxyHost= -Dhttps.proxyPort= -jar webin-cli-1.6.0.jar -test -context genome -userName -password -inputDir -outputDir -validate -manifest -submit

I get a ConnectionTimeoutException at the point it tries to verify the version of cli

raskoleinonen commented 5 years ago

Could you please check if you can curl successfully to the version check endpoint with and without --proxy option:

https://www.ebi.ac.uk/ena/submit/drop-box/check_version/cli/1.6.0

This should return "true".

seretol commented 5 years ago

Tks for your reply.

Yes, I tried curl and it works fine. We can actually submit using the programmatic method (ie curl).

I could not rebuild the whole project locally, but was able to build just enough to execute the version check independently

In 1.6.0, I isolated the Version class and I cannot make it work behind proxies. However, in 1.6.1 there was a significant change, and in this one, I isolated the VersionService and this one seems to work fine behind proxies Note that there is no download of 1.6.1 so I could not check the full webin cli.

So I think it’s an issue with the underlying libraries used to make the http get calls in 1.6.0 (and prior, I tried 1.5.4 and that didn’t work either)

Best,

-oli

From: Rasko Leinonen notifications@github.com Reply-To: enasequence/webin-cli reply@reply.github.com Date: Wednesday, 20 February 2019 at 16:03 To: enasequence/webin-cli webin-cli@noreply.github.com Cc: Olivier Seret os7@sanger.ac.uk, Author author@noreply.github.com Subject: Re: [enasequence/webin-cli] Issue running behind proxy (#9) [EXT]

Could you please check if you can curl successfully to the version check endpoint with and without --proxy option:

https://www.ebi.ac.uk/ena/submit/drop-box/check_version/cli/1.6.0https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ebi.ac.uk_ena_submit_drop-2Dbox_check-5Fversion_cli_1.6.0&d=DwMCaQ&c=D7ByGjS34AllFgecYw0iC6Zq7qlm8uclZFI0SqQnqBo&r=ozJK7FMbK41bJjVG0vBSVQ&m=7E0vW21t7WP0xWwREVYpUJSr9F0bdWW8OM2LJ_zm2l0&s=kLDRwY-kuYJgVHfjcB6LqlA-G4jpy28vP5IYtveR42M&e=

This should return "true".

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_enasequence_webin-2Dcli_issues_9-23issuecomment-2D465640713&d=DwMCaQ&c=D7ByGjS34AllFgecYw0iC6Zq7qlm8uclZFI0SqQnqBo&r=ozJK7FMbK41bJjVG0vBSVQ&m=7E0vW21t7WP0xWwREVYpUJSr9F0bdWW8OM2LJ_zm2l0&s=Fvspn-fhoctIIy6yOa9pImAduvGHcSqwaP8ncSku2qA&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AshWMr18qgrHWQAkXVPM7Q8MEHK6EddPks5vPXHIgaJpZM4bFctu&d=DwMCaQ&c=D7ByGjS34AllFgecYw0iC6Zq7qlm8uclZFI0SqQnqBo&r=ozJK7FMbK41bJjVG0vBSVQ&m=7E0vW21t7WP0xWwREVYpUJSr9F0bdWW8OM2LJ_zm2l0&s=nbYNfFyBBpKv1m8YzoGmEhftbsymF68jnwaNIBNvT5M&e=.

-- The Wellcome Sanger Institute is operated by Genome Research Limited, a charity registered in England with number 1021457 and a company registered in England with number 2742969, whose registered office is 215 Euston Road, London, NW1 2BE.

raskoleinonen commented 5 years ago

Ok, good to know that 1.6.1 worked. We are preparing to release 1.7.0 today or tomorrow that will have the same version checking code as we had in 1.6.1. We had to remove 1.6.1 because it had a fatal genome assembly validation bug (now fixed for 1.7.0 deployment).

raskoleinonen commented 5 years ago

We have now released Webin-CLI 1.7.0:

https://github.com/enasequence/webin-cli/releases/tag/v1.7.0

Could you please give it a try?

seretol commented 5 years ago

Problem solved! Many thanks.

Best,

-oli

From: Rasko Leinonen notifications@github.com Reply-To: enasequence/webin-cli reply@reply.github.com Date: Thursday, 21 February 2019 at 09:04 To: enasequence/webin-cli webin-cli@noreply.github.com Cc: Olivier Seret os7@sanger.ac.uk, Author author@noreply.github.com Subject: Re: [enasequence/webin-cli] Issue running behind proxy (#9) [EXT]

We have now released Webin-CLI 1.7.0:

https://github.com/enasequence/webin-cli/releases/tag/v1.7.0https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_enasequence_webin-2Dcli_releases_tag_v1.7.0&d=DwMCaQ&c=D7ByGjS34AllFgecYw0iC6Zq7qlm8uclZFI0SqQnqBo&r=ozJK7FMbK41bJjVG0vBSVQ&m=trm-nV0XcHX3d99lK3Fk8MuBp74mlvOGMGdjAVAYWU8&s=kGLW0njx6T7xOZOOwfEiuN99ax4_W8wTDGOjkLobXmI&e=

Could you please give it a try?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_enasequence_webin-2Dcli_issues_9-23issuecomment-2D465918533&d=DwMCaQ&c=D7ByGjS34AllFgecYw0iC6Zq7qlm8uclZFI0SqQnqBo&r=ozJK7FMbK41bJjVG0vBSVQ&m=trm-nV0XcHX3d99lK3Fk8MuBp74mlvOGMGdjAVAYWU8&s=TlnSBi5z9FARC9-GvYCBS4_1OJ9a5w9ChjspzOs5Cq4&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AshWMq-2D2iAVYYojh7K2zpoqVTHSusrUqks5vPmE4gaJpZM4bFctu&d=DwMCaQ&c=D7ByGjS34AllFgecYw0iC6Zq7qlm8uclZFI0SqQnqBo&r=ozJK7FMbK41bJjVG0vBSVQ&m=trm-nV0XcHX3d99lK3Fk8MuBp74mlvOGMGdjAVAYWU8&s=bnUPIkP1cV6ldcbJdZgrmFNvQmJxQaq33eAfRcoQAtQ&e=.

-- The Wellcome Sanger Institute is operated by Genome Research Limited, a charity registered in England with number 1021457 and a company registered in England with number 2742969, whose registered office is 215 Euston Road, London, NW1 2BE.