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

DURACLOUD-1292: Updates dependency versions #145

Merged
merged 1 commit into from Jul 21, 2021
Merged

Conversation

bbranan
Copy link
Member

@bbranan bbranan commented Jul 16, 2021

JIRA Ticket: https://duracloud.atlassian.net/browse/DURACLOUD-1292

What does this Pull Request do?

Updates dependencies used by DuraCloud as well as the Management Console, Mill, and Snapshot projects.

How should this be tested?

There should be no functional changes. If the applications build, deploy, pass tests, and work as normal, then these changes were successful.

Additional Notes:

A change to the default stream copy buffer in commons-io caused the chunking process to fail. To resolve this I set the buffer explicitly to the old value (4096) in the IOUtil utility class and updated code that had been using the commons-io class directly to use IOUtil instead.

Interested parties

@duracloud/committers

- Dependency updates to support all DuraCloud modules
- Maintains the default buffer size from common-io IOUtils.copy() prior to version 2.7 where it was 4096. Allowing this to raise to 8192 (the new default) causes failures when performing file chunking.
- Updates AWS client creation to allow discovery of AWS region using the standard provider chain
Copy link
Member

@dbernstein dbernstein left a comment

Choose a reason for hiding this comment

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

Looks good and builds.

@dbernstein dbernstein merged commit 882d048 into develop Jul 21, 2021
@dbernstein dbernstein deleted the duracloud-1292 branch July 21, 2021 14:31
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.

None yet

2 participants