12 Commits (main)

Author	SHA1	Message	Date
Matthieu MOREL	f34ad6c337	chore: fix several checks issues from testifylint ... Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	4 days ago
kimsm28	503acff975	fix(registry): resolve golangci-lint issues in token-auth tests ... - Use (*net.ListenConfig).Listen with the test context instead of net.Listen (noctx). - Use suite.NoError instead of suite.Nil for the error check (testifylint). - Drop the unnecessary leading blank lines in the scope test funcs (whitespace). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: kimsm28 <sm28.kim@samsung.com>	1 week ago
kimsm28	3cbee7e935	fix(registry): use plain-http registry in token-auth scope test ... The scope test configured a TLS registry while advertising an http token realm. oras refuses to send credentials to an http token realm when the registry itself was contacted over https, so the auth server was never reached and the test timed out ("timeout waiting for auth request"). Use a plain-http registry so the registry and token-realm schemes match; the scope carried in the token request is what this test verifies, and that is independent of TLS. Also make the auth handler's channel send non-blocking so that a client retry can never block the handler and stall the push/pull flow, which addresses the review comment about a potential deadlock. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: kimsm28 <sm28.kim@samsung.com>	1 week ago
kimsm28	9b7a70f0ca	test: improve client_scope_test.go to avoid data races and brittle assertions ... - Fix data race by using channel instead of shared variable for requestURL - Make assertions more robust by parsing URL and checking query parameters instead of exact string matching Signed-off-by: kimsm28 <sm28.kim@samsung.com>	1 week ago
kimsm28	1f0a728496	fix registry test failures by adjusting DockerRegistryHost and auth server listener management ... - Change DockerRegistryHost to use 127.0.0.1 for HTTP tests and helm-test-registry for TLS tests to match certificate hostname - Add defer ln.Close() to prevent resource leak in token auth server setup - Restore requestURL variable and assertion in test body instead of handler to avoid potential race condition Signed-off-by: kimsm28 <sm28.kim@samsung.com>	1 week ago
kimsm28	8883c9cc70	fix variable naming requestUrl -> requestURL ... Signed-off-by: kimsm28 <sm28.kim@samsung.com>	1 week ago
kimsm28	530e728ea9	fix typo, remove unnecessary code, fix to avoid to use the assertion in http hanlder ... Signed-off-by: kimsm28 <sm28.kim@samsung.com>	1 week ago
kimsungmin1	26e5071f66	change suite.Nil, suite.NotNill to more proper function(suite.NoError, suite.Error) ... Signed-off-by: kimsungmin1 <sm28.kim@samsung.com>	1 week ago
kimsungmin1	8c8d6e3262	change client_scope_test.go to use httptest ... Signed-off-by: kimsungmin1 <sm28.kim@samsung.com>	1 week ago
kimsungmin1	9d77ccd46d	fix typo ... Signed-off-by: kimsungmin1 <sm28.kim@samsung.com>	1 week ago
kimsungmin1	319e06cda3	add newline in license header ... Signed-off-by: kimsungmin1 <sm28.kim@samsung.com>	1 week ago
kimsungmin1	301e3a12d5	fix scope when helm push to a registry that use token auth ... Signed-off-by: kimsungmin1 <sm28.kim@samsung.com>	1 week ago