Skip to content

Added logic for collecting detailed summary without buildname and bui…#298

Merged
naveenku-jfrog merged 10 commits intojfrog:mainfrom
naveenku-jfrog:fix/RTECO-506-Docker-push-with-detailed-summary-issue
Nov 10, 2025
Merged

Added logic for collecting detailed summary without buildname and bui…#298
naveenku-jfrog merged 10 commits intojfrog:mainfrom
naveenku-jfrog:fix/RTECO-506-Docker-push-with-detailed-summary-issue

Conversation

@naveenku-jfrog
Copy link
Copy Markdown
Collaborator

@naveenku-jfrog naveenku-jfrog commented Oct 30, 2025

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • Appropriate label is added to auto generate release notes.
  • I used gofmt for formatting the code before submitting the pull request.
  • PR description is clear and concise, and it includes the proposed solution/fix.

what: Added logic for collecting detailed summary without buildName and buildNumber in jf docker push command.
How:
There are 2 cases:

  1. If --detailed-summary is provided and Xray security policy is enabled to prevent unscanned artifacts downloads:
    -> In this case, we are just providing the basic layers uploaded by the getting layers info from search functionality. In this case, we are not checking contents of manifest.json.

  2. If --detailed-summary is provided and no Xray security policy is enabled:
    -> This case is simply checking manifest.json's contents to verify the layers and its checksum as previously implemented solution.

In both above cases, we are not involving the collecting build info details.

Depend on any other PRs: No

Testing: Refer Attached screenshot
Screenshot 2025-11-08 at 8 37 08 PM
Screenshot 2025-11-08 at 8 52 04 PM

@naveenku-jfrog naveenku-jfrog added the bug Something isn't working label Oct 30, 2025
@itsmeleela
Copy link
Copy Markdown
Collaborator

@naveenku-jfrog Please share a screen recording of how command working here!

@github-actions
Copy link
Copy Markdown
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


@naveenku-jfrog naveenku-jfrog merged commit a7b8b19 into jfrog:main Nov 10, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants