issue #54: uvloop ssl protocol transport assertion

[zt50tz]

Add uvloop ssl protocol transport class as possible transport class.

[zt50tz]

Issue #55 also refers to this.

[pep8speaks]

Hello @zt50tz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-03-13 19:39:06 UTC

[Fatal1ty]

Hey @zt50tz

I think we can simply drop assertion here. It was introduced in this commit just for typing purposes.

[Fatal1ty]

I made a fix in #58

[zt50tz]

Hi! I thought about it. If it is possible — of course ;) My thoughts were — if assertion exists we admit that variable can be instance of some wrong type (not transport). So I added this option just because i don’t know all the code perfectly. After this change library will start working in fastapi library that uses uvloop by default. I think that is great.

…

On 17 Mar 2024, at 16:22, Alexander Tikhonov ***@***.***> wrote: Hey @zt50tz <https://github.com/zt50tz> I think we can simply drop assertion here. It was introduced in this <df2490f> commit just for typing purposes. — Reply to this email directly, view it on GitHub <#56 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AH6FSK655XC4FDG3YSS5C3DYYWKKPAVCNFSM6AAAAABEUY4HYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSGQ3DONZQHE>. You are receiving this because you were mentioned.

[Fatal1ty]

My thoughts were — if assertion exists we admit that variable can be instance of some wrong type (not transport). So I added this option just because i don’t know all the code perfectly.

I think duck typing is the best choice when we deal with uvloop :D Anyway, thank you for bringing my attention to this, I'm going to prepare a new release soon.