Skip to content
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

Breeze.server and FIPS #131

Open
p-kos opened this issue Mar 13, 2022 · 8 comments
Open

Breeze.server and FIPS #131

p-kos opened this issue Mar 13, 2022 · 8 comments

Comments

@p-kos
Copy link

p-kos commented Mar 13, 2022

The new policies FIPS makes breeze server stop working

@steveschmitt
Copy link
Member

I thought those settings only controlled encryption. Any idea why they make breeze server stop working?

@p-kos
Copy link
Author

p-kos commented Mar 14, 2022

Microsoft does not allow Encryption used in breeze server.

@steveschmitt
Copy link
Member

Is there some sort of error message, or other information that will help us fix the problem?

@steveschmitt
Copy link
Member

Any clue you can give us about how to fix the problem?

@p-kos
Copy link
Author

p-kos commented Apr 27, 2023

What I did, is downloaded the code, fix in my local and compile it.

@steveschmitt
Copy link
Member

steveschmitt commented Apr 27, 2023

Great! What changes did you make to fix it?

I changed my Local Security Policy setting System cryptography: Use FIPS compliant algorithms for encryption, hashing and signing to Enabled. But I still don't get any errors. What else would I need to do to cause the error (so I can tell when it is fixed)?

@lncrowe
Copy link

lncrowe commented Aug 6, 2024

The issue is with using SHA1 to create a string hash, and SHA1 is not FIPS compliant. I saw this problem by making a projection query from the client using a select to only return a single specific field. I needed it in the older AspNet code, so I replaced System.Security.Cryptography.SHA1Managed() with a slightly different algorithm using SHA256.Create(). But I saw that SHA1 is also used in DynamicTypeInfo.cs, which will probably have the same problem.

@steveschmitt
Copy link
Member

Aha! Thank you for the insight.

I still don't know how to reproduce the error, but I found the spot in DynamicTypeInfo.CalcStringHash() that uses SHA1, so I can change it there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants