Skip to content
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

Respect IDE HTTP Proxy settings #685

Closed
kiiadi opened this issue Dec 5, 2018 · 16 comments
Closed

Respect IDE HTTP Proxy settings #685

kiiadi opened this issue Dec 5, 2018 · 16 comments
Labels
feature-request New feature or enhancement. May require GitHub community feedback.
Milestone

Comments

@kiiadi
Copy link
Contributor

kiiadi commented Dec 5, 2018

Calls to AWS via the AWS clients should respect the IDE-wide HTTP settings.

We will need to create our own implementation of the SdkHttpClient, the JetBrains framework has a com.intellij.util.net.HttpConfigurable that we can use to create a HttpUrlConnection (which includes IDE proxy settings if present). We should switch from the AWS ApacheHttpClient (and remove the dependency from our dependency closure), to UrlConnectionHttpClient and then update the AWS SDK for Java so that it can accept a proxy configuration per request.

Also need to update software.aws.toolkits.core.utils.RemoteResourceResolver

@abrooksv abrooksv added the feature-request New feature or enhancement. May require GitHub community feedback. label Dec 5, 2018
@abrooksv
Copy link
Contributor

SDK change was released in 2.2.0

@onedaywillcome1
Copy link

I downloaded aws toolkit in intellij plugins. Even though proxy is set in intellij network settings, toolkit doesn't connect to aws lambda or cloudformation. What is the situation of this issue? @abrooksv

image

image

@abrooksv
Copy link
Contributor

abrooksv commented Jan 2, 2019

We have not implemented it yet, this is a feature request.

When attempting the change we found a bug in the SDK that will need to be fixed first. aws/aws-sdk-java-v2#975

@abrooksv
Copy link
Contributor

abrooksv commented Jan 2, 2019

Bugfix against SDK: aws/aws-sdk-java-v2#989

@abrooksv
Copy link
Contributor

abrooksv commented Jan 2, 2019

Another fix in Java SDK: aws/aws-sdk-java-v2#990

@abrooksv
Copy link
Contributor

abrooksv commented Jan 9, 2019

Code changes have been made and hidden behind a feature flag to opt-in. There are known issues since the SDK issues have not been merged.

#735

@abrooksv
Copy link
Contributor

abrooksv commented Feb 14, 2019

Tested the proxy integration does work but UrlConnectionHttpClient fails with signature validation issues against some services, SDK issue: aws/aws-sdk-java-v2#1078

@bikerp
Copy link

bikerp commented Feb 26, 2019

I'm facing the same issue. We use corporate proxy with authentication and the AWS Toolkit cannot connect to the AWS

@was-av
Copy link

was-av commented Mar 15, 2019

I'm facing the same issue too.

@Sipsip
Copy link

Sipsip commented Mar 19, 2019

I have the same issue. Is there a workaround? I'm not very familiar with proxy-settings in general

@abrooksv
Copy link
Contributor

Trying a different approach: aws/aws-sdk-java-v2#1154

@abrooksv
Copy link
Contributor

abrooksv commented Apr 2, 2019

Proxy support has been re-written and showing good results. Will stay behind an experimental feature flag until we get some real-world users.

@abrooksv abrooksv added this to the Sprint 14 milestone Apr 2, 2019
@abrooksv
Copy link
Contributor

abrooksv commented Apr 4, 2019

Experimental support landed in 1.3-EAP.2019.04.04.

@onedaywillcome1 @waSRull @bikerp If you would like to test it with your setup, please enable EAP builds (https://github.com/aws/aws-toolkit-jetbrains#eap-builds) and then enable the feature through the experimental actions system.

Screen Shot 2019-04-04 at 9 45 22 AM
Then enable the aws.toolkit.useProxy feature and restart.
Screen Shot 2019-04-04 at 9 50 09 AM

@netgitdom
Copy link

Works for me through our corporate proxy, thanks.

@bikerp
Copy link

bikerp commented Apr 8, 2019

@abrooksv Thanks for the update. It works.

@zhangzhx
Copy link
Contributor

This has been released in version 1.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or enhancement. May require GitHub community feedback.
Projects
None yet
Development

No branches or pull requests

8 participants