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

Use of legacy library commons-httpclient:commons-httpclient:3.1 #2691

Open
1 of 2 tasks
peterhaochen47 opened this issue Jan 25, 2024 · 3 comments · Fixed by #2826
Open
1 of 2 tasks

Use of legacy library commons-httpclient:commons-httpclient:3.1 #2691

peterhaochen47 opened this issue Jan 25, 2024 · 3 comments · Fixed by #2826

Comments

@peterhaochen47
Copy link
Member

peterhaochen47 commented Jan 25, 2024

This library is very old and needs to be replaced.

Some examples of usage in our code:

In ExternalOAuthAuthenticationFilter.java

import org.apache.commons.httpclient.util.URIUtil;
...
final String origin = URIUtil.getName(String.valueOf(request.getRequestURL()));

And in TokenMvcMockTests.java in the invalidScopeErrorMessageIsNotShowingAllClientScopes and
invalidScopeErrorMessageIsNotShowingAllUserScopes test methods, for the same import.

See further discussion here: #2689 (comment)

Tasks

@cf-gitbot
Copy link

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

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

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

@strehle
Copy link
Member

strehle commented Apr 11, 2024

Will remove UAA usages, but the external dependency still requires it, see dep. tree

image

@strehle strehle self-assigned this Apr 11, 2024
@strehle strehle linked a pull request Apr 11, 2024 that will close this issue
@strehle strehle removed their assignment Apr 11, 2024
@strehle strehle reopened this Apr 13, 2024
@cf-gitbot
Copy link

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

3 participants