Skip to content

Conversation

@strehle
Copy link
Member

@strehle strehle commented Oct 4, 2017

our code scanner maked this library as problem, because of
CVE-2015-3250

There is no direct usage of problematic code,
but update prevents any issues here

our code scanner maked this library as problem, because of
	CVE-2015-3250

There is no direct usage of problematic code,
but update prevents any issues here
@cfdreddbot
Copy link

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.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/151683806

The labels on this github issue will be updated when the story is started.

@coveralls
Copy link

coveralls commented Oct 4, 2017

Coverage Status

Coverage remained the same at 83.454% when pulling 469f992 on strehle:update/ldap into 12e9e59 on cloudfoundry:develop.

1 similar comment
@coveralls
Copy link

coveralls commented Oct 4, 2017

Coverage Status

Coverage remained the same at 83.454% when pulling 469f992 on strehle:update/ldap into 12e9e59 on cloudfoundry:develop.

@strehle
Copy link
Member Author

strehle commented Oct 26, 2017

hi @fhanik thanks for the other updates in 4.7.1, but is there a reason why you missed this one ?

@fhanik
Copy link
Contributor

fhanik commented Oct 26, 2017

@strehle I believe @sreetummidi and @tnwang may just have missed it.

I'll merge this for develop. 4.7.1 is not affected, we're not using the ApacheDS for very much.

@fhanik fhanik merged commit 20d0a74 into cloudfoundry:develop Oct 26, 2017
@strehle strehle deleted the update/ldap branch October 26, 2017 18:26
@coveralls
Copy link

coveralls commented Oct 26, 2017

Coverage Status

Coverage decreased (-1.5%) to 84.62% when pulling 26d4478 on strehle:update/ldap into e85ffc5 on cloudfoundry:develop.

@cf-gitbot cf-gitbot added delivered accepted Accepted the issue and removed delivered labels Oct 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted Accepted the issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants