Description

fancidev

opened on Oct 11, 2022

Issue body actions

The test_sendfile_fallback_close_peer_in_the_middle_of_receiving test in test.test_asyncio.test_sendfile.SelectEventLoopTests fails with error “OSError: [Errno 41] Protocol wrong type for socket” intermittently on macOS.

I have seen this error multiple times across Python 3.8-3.11rc1 CI test runs on GitHub Actions. For a recent example, see https://github.com/fancidev/qtinter/actions/runs/3219024415/jobs/5263887528 (this is a custom build run, but the test steps involves no custom code)

This is in fact nothing new — a similar issue was reported and fixed in #88395

Maybe a similar fix to the test case would suit.

Activity

1. fancidev

   added

   type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

   on Oct 11, 2022
2. fancidev commented on Oct 11, 2022

   

   fancidev

   on Oct 11, 2022

   ContributorAuthor

   More actions

   Another CI test run with the same error (Python 3.7):
   Python-3.7-PySide2-macos-latest
3. sobolevn

   added

   topic-asyncio

   on Oct 12, 2022
4. ezio-melotti

   added this to asyncioon Oct 12, 2022
5. ezio-melotti

   moved this to Todo in asyncioon Oct 12, 2022
6. sobolevn commented on Oct 12, 2022

   

   sobolevn

   on Oct 12, 2022

   Member

   More actions

   Can you please try the latest version? Will it still fail for you?
   Refs #97546
7. gvanrossum commented on Oct 12, 2022

   

   gvanrossum

   on Oct 12, 2022

   Member

   More actions

   If it does still fail in 3.11 or main, could you suggest a PR? We're short of experts in this area for sure! And I think our main SSL expert works primarily (only?) on Linux.
8. akulakov commented on Oct 14, 2022

   

   akulakov

   on Oct 14, 2022

   Contributor

   More actions

   I could not reproduce locally on MacOS 11.5.2 M1, I did a few hundred test runs on both Py 3.9 and latest dev version.
9. fancidev commented on Oct 16, 2022

   

   fancidev

   on Oct 16, 2022

   ContributorAuthor

   More actions

   @sobolevn: The issue still exists in recent code -- here is a CI run error example on 3.11.0-rc1: https://github.com/fancidev/qtinter/actions/runs/3219024415/jobs/5263887528#logs

   @gvanrossum: I proposed a PR just now. Thanks for reviewing!

   @akulakov: Neither can I reproduce the issue on my dev machine, though I've seen it a few times on CI runs. I suppose the load on CI server makes the issue easier to occur.
10. gvanrossum

    added a commit that references this issue on Oct 17, 2022

    gh-98174: Handle EPROTOTYPE under macOS in test_sendfile_fallback_clo…

    3e82ad0
11. miss-islington

    added 3 commits that reference this issue on Oct 17, 2022

    pythongh-98174: Handle EPROTOTYPE under macOS in test_sendfile_fallba…

    8500564

    pythongh-98174: Handle EPROTOTYPE under macOS in test_sendfile_fallba…

    29f4a44

    gh-98174: Handle EPROTOTYPE under macOS in test_sendfile_fallback_clo…

    3adf234
12. carljm

    added a commit that references this issue on Oct 17, 2022

    Merge branch 'main' into typewatch

    cdbe93b
13. gvanrossum

    added a commit that references this issue on Oct 18, 2022

    [3.11] gh-98174: Handle EPROTOTYPE under macOS in test_sendfile_fallb…

    0bba980
14. kumaraditya303

    closed this as completedon Oct 21, 2022
15. Repository owner moved this from Todo to Done in asyncioon Oct 21, 2022

Issue actions