Repository navigation
fix(pro): packages upload warns when Jamf Pro never hashes a large package - #458
Merged
neilmartin83 merged 2 commits intoOct 29, 2026
Conversation
This was referenced Oct 25, 2026
ktn-jamf
force-pushed
the
fix/package-upload-unverified-hash-stacked
branch
from
October 26, 2026 22:43
6242e04 to
5d81da6
Compare
…ckage Jamf Pro computes no hash for some large packages (seen at 2 GB) even after cloudTransferStatus reaches READY, and a replace clears the old hash without writing a new one. The verify poll then ran its full 10 minutes and ended on a generic "timed out" warning. Once the package is READY at the new file's size with no matching hash, allow a 45s grace and report the upload as unverified. A replace where old and new sizes are identical says the replace cannot be confirmed by size. A hash mismatch still fails as before.
A 300 MB package was verified on the first poll after upload, both on create and on replacing a 2 GB package, so 45s was far longer than a hash that is going to arrive needs. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
neilmartin83
force-pushed
the
fix/package-upload-unverified-hash-stacked
branch
from
October 28, 2026 18:17
5d81da6 to
35785e7
Compare
grahampugh
approved these changes
Oct 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Jamf Pro computes no hash for some large packages even after
cloudTransferStatusisREADY(reported in grahampugh/jamf-upload#418). Wire-checked on a Jamf Pro tenant reporting MD5 only:md5populated within the first pollsREADY, every hash field empty, never filledmd5kept ~6 min during transfer, then cleared whensizeflipped; no new hashBefore this change
pro packages uploadpolled for the full 10 minutes and ended on a generic "timed out" warning, exit 0.Change
READYat the new file's size and no matching hash is present, allow a 20s grace (verifyGrace), then report the upload as unverified instead of waiting outverifyTimeout.previousHashesrecords the record's priorsize;fileHashescarries the local byte length.Stacked
Based on
feat/select-extra-columns(#455) so thetermSafe(err)change on the sameWARNING:line in #453 does not conflict. Merge after #455; the diff here is onlypro_packages_upload.goand its test.Testing
TestVerifyPackageUpload_UnverifiedWhenREADYAtTheRightSizecovers the three cases and asserts the wait ends at the grace, not the timeout.go test -count=1 ./internal/commands,make lint(0 issues),make fmtclean.🤖 Generated with Claude Code