Skip to content

Fix backend compile break after Jackson dependency upgrades#508

Open
JuliusHenke with Copilot wants to merge 3 commits into
renovate/backend-minorfrom
copilot/fix-backend-build-errors
Open

Fix backend compile break after Jackson dependency upgrades#508
JuliusHenke with Copilot wants to merge 3 commits into
renovate/backend-minorfrom
copilot/fix-backend-build-errors

Conversation

Copilot AI commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

The backend started failing at :compileJava on PR #494 after dependency updates because PropertyNamingStrategy.SNAKE_CASE is no longer available in the upgraded Jackson line. The failure occurred in descriptor JSON mappers used by index and onionperf parsing.

  • Jackson API compatibility update

    • Replaced deprecated/removed PropertyNamingStrategy usage with PropertyNamingStrategies in the two failing Java classes.
    • Updated imports accordingly to match current Jackson APIs.
  • Touched parser components

    • org.torproject.descriptor.index.IndexNode
    • org.torproject.descriptor.onionperf.ParsedOnionPerfAnalysis
import com.fasterxml.jackson.databind.PropertyNamingStrategies;

new ObjectMapper()
    .setPropertyNamingStrategy(PropertyNamingStrategies.SNAKE_CASE);

@JuliusHenke JuliusHenke marked this pull request as ready for review July 8, 2026 16:04
Copilot AI review requested due to automatic review settings July 8, 2026 16:04

Copilot AI left a comment

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@JuliusHenke

Copy link
Copy Markdown
Member

@copilot the backend tests still fail. Fix remaining issues.

Copilot AI requested a review from JuliusHenke July 8, 2026 16:14
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