Skip to content

[DPC-5384] Me/dpc 5384 dw upgrade#3004

Open
MEspositoE14s wants to merge 8 commits into
mainfrom
me/dpc-5384-DW-upgrade
Open

[DPC-5384] Me/dpc 5384 dw upgrade#3004
MEspositoE14s wants to merge 8 commits into
mainfrom
me/dpc-5384-DW-upgrade

Conversation

@MEspositoE14s
Copy link
Copy Markdown
Contributor

🎫 Ticket

https://jira.cms.gov/browse/DPC-5384

🛠 Changes

Bumped DropWizard from 4.x to 5.0.1 and updated code where necessary.

ℹ️ Context

  • DropWizard 4.x had reached end of life.
  • Jetty 11 was accumulating Snyk alerts that are only fixed in 12.x, and Jetty 12 is only supported in DropWizard 5.

🧪 Validation

CI passes and services can start and run.

@MEspositoE14s MEspositoE14s requested a review from a team as a code owner May 11, 2026 14:24
@lukey-luke
Copy link
Copy Markdown
Contributor

Looks good to me! Rails dependencies seem to be causing issues with the pipelines still, but all java workflows are passing ✅

@MethodSource("downloadArgs")
void canDownloadFiles(boolean compressFile, Map<String, String> requestHeaders) throws IOException {
String testData = "test data".repeat(500);
String testData = "test data".repeat(50);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I reduced this because the test is still occasionally failing due to resource limitations.

Copy link
Copy Markdown
Contributor

@ashley-weaver ashley-weaver left a comment

Choose a reason for hiding this comment

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

🙌

enabled: true
schemaValidation: false
schematronValidation: false
schematronValidation: false # Schematron has been deprecated, we should remove this in the future.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

do we have a ticket for this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just created DPC-5450.

@MEspositoE14s MEspositoE14s requested a review from a team May 13, 2026 18:26
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.

3 participants