Bug/missing cryptography mysql#318
Conversation
|
@vindex10 Thank you for submitting your contribution! |
|
Thank you! It seems tests for mysql passed :) |
|
@vindex10 Indeed the mysql tests are succeeding. I will have look at the other failing tests |
|
Thanks for having a look into the other failing tests. Could you also rebuild the requirements files by running |
|
DISREGARD. I had to align with the master branch, now it is in progress didnt work out of the box |
|
#331, which adds a workflow to build requirements files, |
d2af4ae to
6c5862e
Compare
|
local build failed. @tillahoffmann please allow workflows :) I rebased on top of #331 |
6c5862e to
33c6be5
Compare
|
@tillahoffmann Hi! I managed to run |
|
@tillahoffmann all checks have passed! 🎉 |
2521db0 to
32cba7a
Compare
|
@tillahoffmann please allow the workflows again, I resolved the conflicts. I will update the requirements with |
|
@tillahoffmann all checks have passed! should we merge before it gained more conflicts? :) |
|
@tillahoffmann done. I hope it is correct :) |
|
Why were the 12 checks cancelled? is it expected? |
|
@tillahoffmann could you please comment? I see only successful or cancelled checks, no failed. |
When installed into a clean environment, testcontainers-mysql requires
cryptographypackage to authenticate with "mock db". This pull requests specifies the requirement forpymysql(which is the reason why cryptography is required):https://github.com/PyMySQL/PyMySQL/blob/b1399c95bcde8ef73cbc3a6d4e8bf767094bbd9e/pyproject.toml#L31
This happens only for mysql:8, mysql:5 doesn't have this issue.