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

dotnet core version #85

Open
1rosehip opened this issue Jan 28, 2017 · 3 comments
Open

dotnet core version #85

1rosehip opened this issue Jan 28, 2017 · 3 comments

Comments

@1rosehip
Copy link

Hi,
Thank you for the repo.
The code could not be compiled in dotnet core because of the following issue:
https://github.com/dotnet/corefx/issues/2881
Please consider adding also dotnet core version file.
Thanks.

@defuse
Copy link
Owner

defuse commented May 2, 2017

I'd be happy to accept a pull request if someone wants to do that, I'm not familiar with dotnet core.

@dan2dev
Copy link

dan2dev commented Jun 7, 2017

Hi,
To use in dotnet core we just need change this:
using (RNGCryptoServiceProvider csprng = new RNGCryptoServiceProvider()) { csprng.GetBytes(salt); }
to this:
using (var rng = RandomNumberGenerator.Create()) { rng.GetBytes(salt); }

@dan2dev
Copy link

dan2dev commented Jun 7, 2017

I created a Pull Request to include the file I made.

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