70aef9f502
gosec: add ignore comments for reported issues that can be ignored
...
```
builder/remotecontext/remote.go:48: G107: Potential HTTP request made with variable url (gosec)
builder/remotecontext/git/gitutils.go:145: G107: Potential HTTP request made with variable url (gosec)
builder/remotecontext/git/gitutils.go:147: G107: Potential HTTP request made with variable url (gosec)
pkg/fileutils/fileutils_test.go:185: G303: File creation in shared tmp directory without using ioutil.Tempfile (gosec)
pkg/tarsum/tarsum_test.go:7: G501: Blacklisted import `crypto/md5`: weak cryptographic primitive (gosec)
pkg/tarsum/tarsum_test.go:9: G505: Blacklisted import `crypto/sha1`: weak cryptographic primitive (gosec)
```
Signed-off-by: Sebastiaan van Stijn <github@gone.nl >
2019-09-18 12:57:43 +02:00
04e2a24a9e
gitutils: add validation for ref
...
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com >
(cherry picked from commit 723b107ca4fba14580a6cd971e63d8af2e7d2bbe)
Signed-off-by: Andrew Hsu <andrewhsu@docker.com >
2019-03-26 22:05:46 +00:00
71672ece9c
Update tests to use gotest.tools 👼
...
Signed-off-by: Vincent Demeester <vincent@sbr.pm >
2018-06-13 09:04:30 +02:00
db857b5d9c
Post migration assertion fixes
...
Signed-off-by: Daniel Nephin <dnephin@docker.com >
2018-03-16 11:03:46 -04:00
242f176825
Automated migration using
...
gty-migrate-from-testify --ignore-build-tags
Signed-off-by: Daniel Nephin <dnephin@docker.com >
2018-03-16 11:03:43 -04:00
6ea4877cff
Add canonical import comment
...
Signed-off-by: Daniel Nephin <dnephin@docker.com >
2018-02-05 16:51:57 -05:00
7bc503344a
gitutils: remove checkout directory on error
...
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com >
2017-12-08 11:58:13 -08:00
e2cc22d076
gitutils: fix checking out submodules
...
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com >
2017-12-07 14:25:19 -08:00
e9831d75e2
Fix shallow git clone in docker-build
...
If the HEAD request fails, use a GET request to properly test if git
server is smart-http.
Signed-off-by: Andrew He <he.andrew.mail@gmail.com >
2017-07-25 13:20:59 -07:00
9450481b7e
Move IsGitTransport() to gitutils
...
This function was only used inside gitutils,
and is written specifically for the requirements
there.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl >
2017-06-26 10:07:04 -07:00
a6cc6cd878
Fix handling of remote "git@" notation
...
`docker build` accepts remote repositories
using either the `git://` notation, or `git@`.
Docker attempted to parse both as an URL, however,
`git@` is not an URL, but an argument to `git clone`.
Go 1.7 silently ignored this, and managed to
extract the needed information from these
remotes, however, Go 1.8 does a more strict
validation, and invalidated these.
This patch adds a different path for `git@` remotes,
to prevent them from being handled as URL (and
invalidated).
A test is also added, because there were no
tests for handling of `git@` remotes.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl >
2017-06-26 10:02:12 -07:00
e907d54fe6
Move pkg/gitutils to remotecontext/git
...
Signed-off-by: Daniel Nephin <dnephin@docker.com >
2017-06-02 16:54:50 -04:00
