Skip to content

Conversation

@reshmifrog
Copy link
Contributor

@reshmifrog reshmifrog commented Jan 6, 2026

  • 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.

Problem: Invalid memory address or nil pointer dereference during jf npm publish
Solution: we were iterating over a list of readers and calling methods on them without checking if any reader was nil, causing a crash. Added a check to skip nil readers instead of crashing.

@reshmifrog reshmifrog added the bug Something isn't working label Jan 6, 2026
@bhanurp bhanurp self-requested a review January 6, 2026 06:04
@reshmifrog reshmifrog merged commit 9bbbed5 into jfrog:main Jan 6, 2026
10 of 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.

2 participants