Skip to content

feat(referrer): improve referrer not found error message #2702

Merged
migmartri merged 2 commits into
chainloop-dev:mainfrom
Piskoo:pfm-4376
Feb 15, 2026
Merged

feat(referrer): improve referrer not found error message #2702
migmartri merged 2 commits into
chainloop-dev:mainfrom
Piskoo:pfm-4376

Conversation

@Piskoo

@Piskoo Piskoo commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

This PR improves the error message when a referrer (artifact/evidence) can't be found during attestation operations.

Closes #2701

Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
@Piskoo Piskoo marked this pull request as ready for review February 3, 2026 14:30
Comment thread app/controlplane/pkg/data/referrer.go Outdated

@migmartri migmartri left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am not sure you are solving the error in the right place, you seem to be doing it at save.

Could you put examples of running the commands before and after?

Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
@Piskoo

Piskoo commented Feb 4, 2026

Copy link
Copy Markdown
Contributor Author

I am not sure you are solving the error in the right place, you seem to be doing it at save.

Could you put examples of running the commands before and after?

Sorry, you are right, it comes from the endpoint used by our builtin chainloop.evidence() rego rule.

$ chainloop att add --value sbom.json
WRN API contacted in insecure mode
INF uploading sbom.json - sha256:bfbb8312c63447567e65f128ac05ddaebf562d072532b37fb412f47bfc32a421
ERR referrer not found
exit status 1

$ chainloop att add --value sbom.json
WRN API contacted in insecure mode
INF uploading sbom.json - sha256:bfbb8312c63447567e65f128ac05ddaebf562d072532b37fb412f47bfc32a421
ERR artifact or piece of evidence with digest sha256:bfbb8312c63447567e65f128ac05ddaebf562d072532b37fb412f47bfc32a421 not found
exit status 1

@Piskoo Piskoo requested a review from migmartri February 4, 2026 13:12
@migmartri migmartri merged commit 2502877 into chainloop-dev:main Feb 15, 2026
13 checks passed
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.

Improve referrer error message to include digest and artifact type

2 participants