Skip to content

Use AES encryption when not on windows - #3884

Merged
forki merged 2 commits into
fsprojects:bugfixfrom
bengobeil:bugfix
Jul 23, 2020
Merged

forki merged 2 commits into
fsprojects:bugfixfrom
bengobeil:bugfix

Conversation

@bengobeil

Copy link
Copy Markdown
Contributor

(cherry picked from commit b22361c)

@bengobeil

Copy link
Copy Markdown
Contributor Author

If we can release this today, it will be very helpful for us. Thanks!

@forki

forki commented Jul 23, 2020

Copy link
Copy Markdown
Member

image

@bengobeil

Copy link
Copy Markdown
Contributor Author

What are the repro steps?

@bengobeil

Copy link
Copy Markdown
Contributor Author

Oh its just doesn't build... one sec

@bengobeil

Copy link
Copy Markdown
Contributor Author

Oh, probably a different lang version on 5.XXX ?

@forki

forki commented Jul 23, 2020

Copy link
Copy Markdown
Member

yes it's old school ;-)

@bengobeil

Copy link
Copy Markdown
Contributor Author

OK, it was a case of implicit yields

@forki

forki commented Jul 23, 2020

Copy link
Copy Markdown
Member

Cool. Will release as soon as I'm back home

@bengobeil

Copy link
Copy Markdown
Contributor Author

Great thanks!

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

Successfully merging this pull request may close these issues.

2 participants