Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 98.48%. Comparing base (4598b89) to head (6c7c0a1).
Report is 2135 commits behind head on master.

@@            Coverage Diff             @@
##           master   #10783      +/-   ##
==========================================
- Coverage   98.50%   98.48%   -0.02%     
==========================================
  Files         155      155              
  Lines       20540    20544       +4     
==========================================
  Hits        20232    20232              
- Misses        308      312       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Thanks for the contribution.

This is not compatible with CPython, which doesn't have this closenotify() method. We need to find a solution that is compatible with CPython.

If you were implementing a Gemini server to run under CPython, how would it work?

I can move this to close(), requiring a CLOSING state before CLOSED. That would make close() a potentially blocking call.

EDIT: looked at cpython, close_notify belongs in shutdown().

This is an automated heads-up that we've just merged a Pull Request
that removes the STATIC macro from MicroPython's C API.

See #13763

A search suggests this PR might apply the STATIC macro to some C code. If it
does, then next time you rebase the PR (or merge from master) then you should
please replace all the STATIC keywords with static.

Although this is an automated message, feel free to @-reply to me directly if
you have any questions about this.

Closing due to inactivity. If this is still of interest feel free to reopen with some more detailed discussion about how CPython handles this case. (Also note that MicroPython now has a tls module which can have incompatibilities with CPython but it would be best to try and make the functionality match CPython.)