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

Remove superflous insensitiviseMaps in Unmarshal methods #666

Merged
merged 1 commit into from
Mar 13, 2019

Conversation

bep
Copy link
Collaborator

@bep bep commented Mar 12, 2019

Fixes #482

@CLAassistant
Copy link

CLAassistant commented Mar 12, 2019

CLA assistant check
All committers have signed the CLA.

Copy link

@xvello xvello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thanks!

@stanislav-bios-baranov
Copy link

👍 Thanks!

@bep bep merged commit 9e56dac into spf13:master Mar 13, 2019
@lnashier
Copy link

lnashier commented Aug 6, 2019

Thanks 👍

@mdelapenya
Copy link

I do not think this is enough, please take a look at #743: first commit changes the tests to have an Uppercased key and tests fail

@wolandtel
Copy link

#664 hasn't been merged so bugs are still here.

@poornachandratejasvi
Copy link

Still facing the same issue.. need a fix

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.

UnmarshalKey function is not safe for concurrent access.
8 participants