Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Submitting snapshot fails on PRs #60

Open
Napfton opened this issue Jan 18, 2024 · 2 comments
Open

Submitting snapshot fails on PRs #60

Napfton opened this issue Jan 18, 2024 · 2 comments

Comments

@Napfton
Copy link

Napfton commented Jan 18, 2024

I have setup the maven-dependecy-submission-action and it works well on the development branch, but fails on any PR from another branch into development with this errors:

Error: Cannot read properties of null (reading 'toString')
Error: TypeError: Cannot read properties of null (reading 'toString')
    at /__w/_actions/advanced-security/maven-dependency-submission-action/v3/node_modules/@github/dependency-submission-toolkit/dist/index.js:506:1
    at Generator.next (<anonymous>)
    at fulfilled (/__w/_actions/advanced-security/maven-dependency-submission-action/v3/node_modules/@github/dependency-submission-toolkit/dist/index.js:395:1)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

/__w/_actions/advanced-security/maven-dependency-submission-action/v3/node_modules/@github/dependency-submission-toolkit/dist/index.js:395
        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
^
Error: Failed to submit snapshot: TypeError: Cannot read properties of null (reading 'toString')
    at /__w/_actions/advanced-security/maven-dependency-submission-action/v3/node_modules/@github/dependency-submission-toolkit/dist/index.js:520:1
    at Generator.next (<anonymous>)
    at fulfilled (/__w/_actions/advanced-security/maven-dependency-submission-action/v3/node_modules/@github/dependency-submission-toolkit/dist/index.js:395:1)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

I already set permissions contents: write which was missing before (worked anyways) but to no avail.

@VOVELEE
Copy link

VOVELEE commented Jan 22, 2024

I am hitting the same

@peter-murray
Copy link
Contributor

What version of the action are you using? The error is coming out of the dependency-submission-toolkit so I need that information to track into the library as to what is not resolving here.

It might be that the commit SHA is pointing to the merged result in the PR or there is a mismatch between the SHA and the ref? Can you provide any additional details on the workflow, trigger and the inputs being provided to the action?

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

No branches or pull requests

3 participants