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

Proxy support does not work when moving from v2 to v4 #1061

Open
RaviVadera opened this issue May 7, 2024 · 2 comments
Open

Proxy support does not work when moving from v2 to v4 #1061

RaviVadera opened this issue May 7, 2024 · 2 comments
Assignees
Labels
bug Something isn't working p1

Comments

@RaviVadera
Copy link

Describe the bug

We are using v2 behind a proxy. Proxy is setup through env vars and not explicitly on the action inputs. We are attempting to move to v4 because of deprecation of Node 16 actions and since Node 20 support is only available on v4. Proxy support on v4 does not work and fails with Error: connect ETIMEDOUT xxx.xxx.xxx.xxx:443.

Expected Behavior

Update to v4 should would without any changes to the action as I do not see that there is any breaking change from changelog.

Current Behavior

Updating action from v2 to v4 fails with Error: connect ETIMEDOUT xxx.xxx.xxx.xxx:443.

Reproduction Steps

  1. Use the v2 action behind a proxy, set the env vars in GH action, the action should be successful.
env:
  http_proxy: xxxx
  https_proxy: xxxx
  no_proxy: xxxx

jobs:
  test:
    name: Test
    runs-on:
      group: xxxx
    environment:
      name: ${{ inputs.environment-name }}
    container:
      image: xxxx

    steps:
      - name: Configure AWS Credentials
        uses: aws-actions/configure-aws-credentials@v2
        with:
          aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
          aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
          aws-region: ${{ vars.AWS_REGION }}
  1. Update the action by just changing the version to v4.
env:
  http_proxy: xxxx
  https_proxy: xxxx
  no_proxy: xxxx

jobs:
  test:
    name: Test
    runs-on:
      group: xxxx
    environment:
      name: ${{ inputs.environment-name }}
    container:
      image: xxxx

    steps:
      - name: Configure AWS Credentials
-        uses: aws-actions/configure-aws-credentials@v2
+        uses: aws-actions/configure-aws-credentials@v4
        with:
          aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
          aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
          aws-region: ${{ vars.AWS_REGION }}

Possible Solution

From the comment at #861 (comment), it looks like there is a change in proxy implementation on v3 which can be the cause of failure. We tried updating from v2 to v3 and can confirm that proxy does not work even on v3.

Additional Information/Context

No response

@RaviVadera RaviVadera added bug Something isn't working needs-triage This issue still needs to be triaged labels May 7, 2024
@tim-finnigan tim-finnigan self-assigned this Jul 3, 2024
@tim-finnigan
Copy link
Contributor

Here is documentation on proxy configuration: https://github.com/aws-actions/configure-aws-credentials?tab=readme-ov-file#proxy-configuration

Have you tried using http-proxy as documented there? What error are you getting?

@tim-finnigan tim-finnigan added response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 5 days. and removed needs-triage This issue still needs to be triaged labels Jul 3, 2024
@RaviVadera
Copy link
Author

@tim-finnigan all infromation included above already. But to reiterate,

We are relying on action to take the proxy from env (as indicated by documentation Additionally this action will always consider the HTTP_PROXY environment variable.), so not configuring http-proxy explicitly.

As for the error,

Proxy support on v4 does not work and fails with Error: connect ETIMEDOUT xxx.xxx.xxx.xxx:443.

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 5 days. label Jul 8, 2024
@kellertk kellertk added p1 and removed p2 labels Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working p1
Projects
None yet
Development

No branches or pull requests

3 participants