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

upgraded jackson libraries to 2.9.10 because 2.4 is out of service #4146

Closed
wants to merge 1 commit into from

Conversation

ajschmidt
Copy link
Contributor

What does this PR (Pull Request) do?

This PR is a replacement for PR #4110 and #4109. I just bumps the version of the Jackson libraries to the latest compatible versions.

  • This PR is not related to any Issue

Which Traffic Control components are affected by this PR?

  • Traffic Router

This is a simple update to the version of the Jackson dependencies responsible for parsing JSON data. It has no effect on documentation.

What is the best way to verify this PR?

If this is a bug fix, what versions of Traffic Control are affected?

Not a bug fix. It can be verified by running the build and the 'verify' testing target. The result should be 'SUCCESS'.

The following criteria are ALL met by this PR

  • This PR includes tests OR I have explained why tests are unnecessary
  • This PR includes documentation OR I have explained why documentation is unnecessary
  • This PR includes an update to CHANGELOG.md OR such an update is not necessary
  • This PR includes any and all required license headers
  • This PR ensures that database migration sequence is correct OR this PR does not include a database migration
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

@asf-ci
Copy link
Contributor

asf-ci commented Nov 23, 2019

Can one of the admins verify this patch?

@ocket8888 ocket8888 closed this Nov 24, 2019
@ajschmidt
Copy link
Contributor Author

Why? I think you meant to merge this one. Its the replacement for 4110 and 4109.

@ocket8888
Copy link
Contributor

My mistake, I thought this was the other dependabot one.

@ocket8888 ocket8888 reopened this Nov 25, 2019
@mitchell852 mitchell852 added Traffic Router related to Traffic Router improvement The functionality exists but it could be improved in some way. labels Nov 25, 2019
@ericholguin
Copy link
Contributor

This was done in #6506

@ericholguin ericholguin closed this Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement The functionality exists but it could be improved in some way. Traffic Router related to Traffic Router
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants