community
/
docker-distribution
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
4,846 Commits 1 Branch 55 Tags 27 MiB
Commit Graph

4846 Commits

Author SHA1 Message Date
Sebastiaan van Stijn dec03ea3d8
update golangci-lint to v1.52 
Removing the "structcheck" and "varcheck" linters as they've been deprecated.

    level=warning msg="[runner] The linter 'structcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused."
    level=warning msg="[runner] The linter 'varcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused."

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 16:04:17 +02:00
Sebastiaan van Stijn ebe9d67446
ignore SA1019: ac.(*accessController).rootCerts.Subjects has been deprecated 
We need to look into this; can we remove it, or is there a replacement?

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 16:04:17 +02:00
Sebastiaan van Stijn 84a85a4048
Ignore SA1019: SplitHostname is deprecated. 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 16:04:17 +02:00
Sebastiaan van Stijn 999527f978
Ignore SA1019: "schema1 is deprecated" linting errors 
We need to use this for backward compatibility.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 16:04:17 +02:00
Sebastiaan van Stijn 3c144f2264
registry/auth/token: fix the surrounding loop is unconditionally terminate 
registry/auth/token/types_test.go:83:3: SA4004: the surrounding loop is unconditionally terminated (staticcheck)
                   return
                   ^

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 16:04:17 +02:00
Sebastiaan van Stijn 4052d269f5
reference, registry: fix loop variable captured by func literal (govet) 
reference/normalize_test.go:274:40: loopclosure: loop variable r captured by func literal (govet)
                                   named, err := ParseNormalizedNamed(r)
                                                                      ^
    reference/normalize_test.go:276:29: loopclosure: loop variable r captured by func literal (govet)
                        t.Fatalf("ref=%s: %v", r, err)
                                               ^
    registry/api/errcode/errors_test.go:45:7: loopclosure: loop variable ec captured by func literal (govet)
                if ec != desc.Code {
                   ^
    registry/api/errcode/errors_test.go:46:66: loopclosure: loop variable ec captured by func literal (govet)
                    t.Fatalf("error code in descriptor isn't correct, %q != %q", ec, desc.Code)
                                                                                 ^
    registry/api/errcode/errors_test.go:49:23: loopclosure: loop variable desc captured by func literal (govet)
                if idToDescriptors[desc.Value].Code != ec {
                                   ^
    registry/api/errcode/errors_test.go:50:80: loopclosure: loop variable desc captured by func literal (govet)
                    t.Fatalf("error code in idToDesc isn't correct, %q != %q", idToDescriptors[desc.Value].Code, ec)
                                                                                               ^
    registry/api/errcode/errors_test.go:53:7: loopclosure: loop variable ec captured by func literal (govet)
                if ec.Message() != desc.Message {
                   ^

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 16:04:08 +02:00
Milos Gajdos c9b844e533
Merge pull request #3885 from thaJeztah/rename_table_tests 
assorted test updates
 2023-05-09 13:58:29 +01:00
Sebastiaan van Stijn f03d966ef7
cloudfront: use consistent names for test-tables, t.Parallel() 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 14:39:58 +02:00
Sebastiaan van Stijn 5301ae14bf
cloudfront: rename vars that collided with type 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 14:00:17 +02:00
Sebastiaan van Stijn 85028a801b
registry/handlers: use consistent names for test-tables 
Also marked assertBlobUploadStateEquals as t.Helper()

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 14:00:17 +02:00
Sebastiaan van Stijn f4acd98865
registry/api/v2: checkTestRouter(): use sub-tests, t.Parallel() 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 14:00:05 +02:00
Sebastiaan van Stijn 2444c3282d
registry/api/v2: use consistent names for test-tables 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:55:16 +02:00
Sebastiaan van Stijn 9266220c2a
registry/auth/token: TestAudienceList_Unmarshal: t.Parallel() 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:55:16 +02:00
Sebastiaan van Stijn 5e67a40e08
registry/auth/token: use consistent names for test-tables 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:55:16 +02:00
Sebastiaan van Stijn f884a079df
registry/api/errorcode: TestErrorCodes: use sub-tests 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:55:11 +02:00
Sebastiaan van Stijn 57f9f31af9
notifications: don't use un-keyed structs 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:38 +02:00
Sebastiaan van Stijn e9ac1728e6
notifications: use consistent names for test tables 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:38 +02:00
Sebastiaan van Stijn f4be328bff
context: TestWithTrace(): inline checkContextForValues, use sub-tests 
checkContextForValues was effectively running sub-tests, but disguided
as a helper (to DRY). While it helped some duplication, rewriting it to
run subtests within a helper also was a bit confusing, so just inline
what it does.

While updating, also run tests with t.Parallel()

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:37 +02:00
Sebastiaan van Stijn 2829f7b7d5
context: use consistent names for test-tables 
Also renamed a var that collided with a type, and marked a helper
as t.Helper()

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:37 +02:00
Sebastiaan van Stijn f238f7dcaa
reference: TestMatch(): use sub-tests 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:37 +02:00
Sebastiaan van Stijn 2819bca991
reference: TestNormalizedSplitHostname(): use sub-tests 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:37 +02:00
Sebastiaan van Stijn fa1d14c513
reference: TestParseAnyReference(): use sub-tests 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:37 +02:00
Sebastiaan van Stijn fcbddfc6ae
reference: use consistent names for test-tables 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-09 13:54:37 +02:00
Wang Yan 8900e90699
Merge pull request #3895 from pluralsh/bug-fix-5001-server-rebase 
Fix the issue that the debug server with port 5001 run twice
 2023-05-05 13:21:55 +08:00
Milos Gajdos 08f7a47bc9
Merge pull request #2766 from glefloch/remove-testdriver 
Remove registry storage testdriver
 2023-05-03 21:40:53 +01:00
David van der Spek 99a8ad00ea fix: rename log to logrus 
Signed-off-by: David van der Spek <vanderspek.david@gmail.com>
 2023-05-03 19:02:03 +02:00
Feng Honglin f0fdaff0a5 Fix the issue that the debug server with port 5001 run twice 
When configuring bugsnag, bugsnag will fork the process, resulting the port 5001 listened twice. The PR fix this error by moving the initialization of prometheus server after the configuration of bugsnag

Signed-off-by: Honglin Feng <tifayuki@gmail.com>
(cherry picked from commit 5a6a2d6ae06453136f5e1cfb5e9efa20c27085d9)
Signed-off-by: Derek McGowan <derek@mcgstyle.net>
Signed-off-by: David van der Spek <vanderspek.david@gmail.com>
 2023-05-03 18:37:32 +02:00
Milos Gajdos 4adbb690c1
Merge pull request #3795 from AdamKorcz/fuzz222 
Fuzzing: Move over two fuzzers from cncf-fuzzing
 2023-05-03 15:59:20 +01:00
AdamKorcz e2a43ec8d3
Fuzzing: Move over two fuzzers from cncf-fuzzing 
Signed-off-by: AdamKorcz <adam@adalogics.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-05-02 23:03:57 +02:00
Milos Gajdos 0ea92a1a8d
Merge pull request #3883 from thaJeztah/reference_update_tests_and_add_benchmark 
reference: rewrite test to use sub-tests, add benchmark
 2023-05-02 20:55:05 +01:00
glefloch 9fb201e23d Remove registry storage testdriver 
Signed-off-by: glefloch <glfloch@gmail.com>
 2023-05-02 16:37:33 +02:00
Milos Gajdos 788598e23f
Merge pull request #3881 from vvoland/handle-forbidden 
registry/errors: Parse http forbidden as denied
 2023-05-02 09:38:36 +01:00
Milos Gajdos 1062ccc7ef
Merge pull request #3882 from thaJeztah/no_dot_imports 
remove dot-imports for gopkg.in/check.v1
 2023-05-01 16:21:11 +01:00
Sebastiaan van Stijn 49e2de2830
reference: add BenchmarkParse 
BenchmarkParse
    BenchmarkParse-10    	   12696	     93805 ns/op	    9311 B/op	     185 allocs/op
    PASS

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-04-29 19:31:21 +02:00
Sebastiaan van Stijn b50c049fc6
reference: TestParseRepositoryInfo: use subtests 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-04-29 18:58:09 +02:00
Sebastiaan van Stijn af36dd698f
reference: TestParseDockerRef: capture test in loop 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-04-29 18:58:01 +02:00
Sebastiaan van Stijn 3fa6d5a33b
remove dot-imports for gopkg.in/check.v1 
Dot-imports were only used in a couple of places, and replacing them
makes it more explicit what's imported.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2023-04-29 17:08:47 +02:00
Paweł Gronowski 5f1df02149
registry/errors: Parse http forbidden as denied 
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
 2023-04-27 19:48:32 +02:00
Milos Gajdos 29b5e79f82
Merge pull request #3742 from sagikazarmark/fix-aud-claim-list 
Accept list of strings in audience claim in token auth
 2023-04-26 18:39:26 +01:00
Milos Gajdos 2fb8dbdeca
Merge pull request #3839 from kirat-singh/feature.azure-sdk-update 
Update Azure SDK and support additional authentication schemes
 2023-04-25 19:35:34 +01:00
Kirat Singh ba4a6bbe02 Update Azure SDK and support additional authentication schemes 
Microsoft has updated the golang Azure SDK significantly.  Update the
azure storage driver to use the new SDK.  Add support for client
secret and MSI authentication schemes in addition to shared key
authentication.

Implement rootDirectory support for the azure storage driver to mirror
the S3 driver.

Signed-off-by: Kirat Singh <kirat.singh@beacon.io>

Co-authored-by: Cory Snider <corhere@gmail.com>
 2023-04-25 17:23:20 +00:00
Milos Gajdos 0c958010ac
Merge pull request #3763 from distribution/multipart-upload-empty-files 
Enable pushing empty blobs
 2023-03-27 10:18:44 +01:00
Milos Gajdos 5fa926a609
Enable pushing empty blobs 
This is an edge case when we are trying to upload an empty chunk of data using
a MultiPart upload. As a result we are trying to complete the MultipartUpload
with an empty slice of `completedUploadedParts` which will always lead to 400
being returned from S3 See: https://docs.aws.amazon.com/sdk-for-go/api/service/s3/#CompletedMultipartUpload
Solution: we upload an empty i.e. 0 byte part as a single part and then append it
to the completedUploadedParts slice used to complete the Multipart upload.

Signed-off-by: Milos Gajdos <milosthegajdos@gmail.com>
 2023-03-27 10:11:07 +01:00
Milos Gajdos e5d5810851
Merge pull request #3847 from aaronlehmann/s3-pagination-loop 
Fix S3 multipart upload pagination loop condition
 2023-02-23 07:28:52 +00:00
Aaron Lehmann 2074688be9 Fix S3 multipart upload pagination loop condition 
The loop that iterates over paginated lists of S3 multipart upload parts
appears to be using the wrong variable in its loop condition. Nothing
inside the loop affects the value of `resp.IsTruncated`, so this loop
will either be wrongly skipped or loop forever.

It looks like this is a regression caused by commit
7736319f2e. The return value of
`ListMultipartUploads` used to be assigned to a variable named `resp`,
but it was renamed to `partsList` without updating the for loop
condition.

I believe this is causing an error we're seeing with large layer uploads
at commit time:

    upload resumed at wrong offset: 5242880000 != 5815706782

Missing parts of the multipart S3 upload would cause an incorrect size
calculation in `newWriter`.

Signed-off-by: Aaron Lehmann <alehmann@netflix.com>
 2023-02-21 20:57:50 -08:00
Milos Gajdos a38eb86fa6
Merge pull request #3845 from distribution/dependabot/go_modules/golang.org/x/net-0.7.0 2023-02-20 14:00:06 +00:00
Milos Gajdos 772cef6b4e
Merge pull request #3736 from aaronlehmann/log-username-on-successful-requests 
Log username on successful requests
 2023-02-19 19:05:35 +00:00
dependabot[bot] 9594fbcfeb
build(deps): bump golang.org/x/net from 0.4.0 to 0.7.0 
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.4.0 to 0.7.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](https://github.com/golang/net/compare/v0.4.0...v0.7.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
 2023-02-17 22:42:11 +00:00
Hayley Swimelar a4d64adabc
Merge pull request #3838 from smorimoto/update-fossa-action 
Update fossa-contrib/fossa-action action to v2
 2023-02-17 14:28:03 -08:00
Milos Gajdos e64b08ada6
Merge pull request #3841 from kirat-singh/feature/aws-ca-bundle 
Support AWS_CA_BUNDLE when talking to the S3 API
 2023-02-17 09:36:20 +00:00