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

GH Actions: improve proxy access check step(s) #891

Open
jrfnl opened this issue Sep 2, 2024 · 0 comments
Open

GH Actions: improve proxy access check step(s) #891

jrfnl opened this issue Sep 2, 2024 · 0 comments

Comments

@jrfnl
Copy link
Member

jrfnl commented Sep 2, 2024

The GH Actions test and quicktest workflows currently contain a "Access localhost on port 9002" step, which is not working correctly.

      - name: Access localhost on port 9002
        run: curl -i http://localhost:9002

Results in:

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
HTTP/1.1 502 Bad Gateway
Server: mitmproxy 10.4.2
Connection: close
Content-Type: text/html
content-length: 231

<html>
<head>
    <title>502 Bad Gateway</title>
</head>
100   231  100   231    0     0  43008      0 --:--:-- --:--:-- --:--:-- 46200
<body>
    <h1>502 Bad Gateway</h1>
    <p>Connection killed: Request destination unknown. Unable to figure out where this request should be forwarded to.</p>
</body>
</html>

We should add the Curl -f flag to fail the step when the connection was not correctly made.

However, as things are, adding the -f flag would currently fail the step, so we first need to do something to improve the step and get it passing properly.

Additionally, we should add a second step to run a similar access test on port 9003.

Lastly, as - even with this failing proxy check - the tests are passing, we should verify that there are sufficient tests for the proxy handling in Requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant