-
Notifications
You must be signed in to change notification settings - Fork 464
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
Fixed tests, build on Arch Linux #416
Conversation
Can you help me understand, if you know, the cause for the issue?
Why? I mean is there a size that is valid for this algorithm? Should it be smaller (idk if it can be) or larger (shall we try)? |
Looks like the provided secret is too short indeed. It must be 128 bit long, or 256 etx. I don't know why it works on other platforms though, maybe they're aren't that reactive/picky. |
I haven't used AutoFixture so I have no idea how to set the key size |
Right, the issue is in AutoFixture that instantiates RSACryptoServiceProvider with a key length or doesn't like. If just |
I was conservative with the changes as it was already running on Linux for versions below .net6 I can try change it so all get same behaviour instead if it works |
830584b
to
f379952
Compare
It seems that just newing up RSACryptoServiceProvider is working for all .net versions so I have updated the code. |
I approved. Please see if you can check it in after the gated build has succeeded. |
Should be alright now :) |
Thank you, sir! |
Will probably be needed for #407
Unless this fix is applied the tests fail with this error when running on arch linux: