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.

Sign up for GitHub

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

Jump to bottom

Uniqueness of Checksums as Idenifiers #9

Closed
metalivedev opened this issue Feb 1, 2013 · 1 comment
Closed

Uniqueness of Checksums as Idenifiers #9

metalivedev opened this issue Feb 1, 2013 · 1 comment

Comments

@metalivedev
Copy link
Contributor

How sure are you of that checksum's "universal uniqueness"?
That seems like a much shorter UUID than any other I've seen, and assuming that a faulty checksum uniquely identifies a container could lead to some terrible collisions.
@shykes
Copy link
Contributor

shykes commented Feb 1, 2013

You're right, the IDs are too short. The UI shows a truncated ID to keep the output readable. The actual checksum is a SHA256 of the full content in tar archive. Going forward that's what we'll use as a canonical identifier.

@shykes shykes closed this as completed Feb 25, 2013
crosbymichael referenced this issue in crosbymichael/docker Nov 8, 2013
@crosbymichael
Merge pull request #9 from crosbymichael/fix-aufs-tests
6d0b3f3 
Fix aufs error at startup
@chrisruffalo chrisruffalo mentioned this issue Feb 12, 2014
Closed
vieux added a commit that referenced this issue Jul 9, 2014
@vieux
Merge pull request #9 from tianon/vieux-hotfix-minor-nits
4634358 
Update RELEASE-CHECKLIST with a few minor nits (periods, whitespace)
proppy pushed a commit to proppy/docker that referenced this issue Jul 24, 2014
@crosbymichael
Merge pull request moby#9 from crosbymichael/add-pull
6005a9e 
Add pull method
cpuguy83 pushed a commit to cpuguy83/docker that referenced this issue Oct 24, 2014
@bfirsh
Merge pull request moby#9 from psftw/host-management
c6d9ad8 
replace os/user with go-homedir
@brain-zhang brain-zhang mentioned this issue Mar 24, 2015
Closed
squaremo added a commit to squaremo/docker that referenced this issue Apr 15, 2015
@squaremo
Merge pull request moby#9 from squaremo/net_driver_plugins
e4394b0 
Net driver plugins
cherrry pushed a commit to cherrry/docker that referenced this issue Feb 11, 2016
@crawford
Merge pull request moby#9 from crawford/listenbuffer
4419fdb 
Revert "Return listenbuffer behavior"
@Snorch Snorch mentioned this issue Oct 21, 2016
Closed
@zhangyongsu zhangyongsu mentioned this issue Jan 8, 2017
Closed
@adaiguoguo adaiguoguo mentioned this issue Sep 1, 2017
Closed
@dazinator dazinator mentioned this issue Aug 18, 2021
Open
thaJeztah pushed a commit to thaJeztah/docker that referenced this issue Jul 8, 2022
@rumpl
Merge pull request moby#9 from vvoland/containerd-store-pull-auth
7d9ebf8 
containerd/pull: Use authorization
thaJeztah added a commit that referenced this issue Oct 18, 2022
@thaJeztah
Merge pull request #9 from moby/update_buildkit
3adff51 
[20.10] vendor moby/buildkit v0.8.3-31-gc0149372
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@shykes @metalivedev