-
Notifications
You must be signed in to change notification settings - Fork 828
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
update json parser #788
update json parser #788
Conversation
Hey strehle! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/155819103 The labels on this github issue will be updated when the story is started. |
@strehle What's the story here? Do you think this change is needed? |
Story is the CVE https://nvd.nist.gov/vuln/detail/CVE-2018-7489 |
Ah, 2.9.5 isn't out yet. I was confused about why you closed this PR but now I understand. We will bump when they release a fix. |
Hey strehle! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/156423446 The labels on this github issue will be updated when the story is started. |
hi @jhamon , update available, therefore please use it for next version |
Co-authored-by: xsuaabot <[email protected]>
https://nvd.nist.gov/vuln/detail/CVE-2018-7489