Do not rely on Docker-Content-Digest when getting manifests by jonesbusy · Pull Request #526 · oras-project/oras-java

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@jonesbusy

Copy link

Collaborator

Description

Fix #522

Testing done

mvn clean install and enable again test

Submitter checklist

  • I have read and understood the CONTRIBUTING guide
  • I have run mvn license:update-file-header, mvn spotless:apply, pre-commit run -a, mvn clean install before opening the PR

@jonesbusy jonesbusy added the bug

Something isn't working

label

Feb 7, 2026

@codecov

Copy link

codecov bot commented

Feb 7, 2026

edited

Loading

Codecov Report

❌ Patch coverage is 92.85714% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.60%. Comparing base (f571253) to head (e7c6b92).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/main/java/land/oras/Registry.java 92.85% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #526      +/-   ##
============================================
+ Coverage     87.85%   88.60%   +0.74%     
+ Complexity      663      662       -1     
============================================
  Files            39       39              
  Lines          2018     2027       +9     
  Branches        231      233       +2     
============================================
+ Hits           1773     1796      +23     
+ Misses          156      140      -16     
- Partials         89       91       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Signed-off-by: Valentin Delaye <jonesbusy@users.noreply.github.com>

@jonesbusy jonesbusy force-pushed the feature/ecr-manifests branch from 3f37bdb to e7c6b92 Compare

February 7, 2026 11:07

@jonesbusy jonesbusy merged commit 25cccf5 into oras-project:main

Feb 7, 2026

8 checks passed

@jonesbusy jonesbusy deleted the feature/ecr-manifests branch

February 7, 2026 11:18

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

Reviewers

No reviews

Assignees

No one assigned

Labels

bug

Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fetching manifest from ECR fails due to missing Docker-Content-Digest header

1 participant

@jonesbusy