From 0f1f4383668c0ec2dad624ae48b3e61f468723aa Mon Sep 17 00:00:00 2001 From: "Jeffrey N. Johnson" Date: Wed, 3 Jun 2026 10:23:29 -0700 Subject: [PATCH 1/3] Bumping some action versions. --- .github/workflows/autotest_prs.yml | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/.github/workflows/autotest_prs.yml b/.github/workflows/autotest_prs.yml index 3ccffbeb..d7d4aa60 100644 --- a/.github/workflows/autotest_prs.yml +++ b/.github/workflows/autotest_prs.yml @@ -7,8 +7,6 @@ on: branches: [ main ] workflow_dispatch: - - jobs: build: @@ -19,14 +17,14 @@ jobs: pull-requests: write steps: - - name: Setting up Go 1.21 - uses: actions/setup-go@v4 + - name: Setting up Go 1.24 + uses: actions/setup-go@v6 with: - go-version: ^1.21 + go-version: ^1.24 id: go - name: Checking out DTS - uses: actions/checkout@v4 + uses: actions/checkout@v5 - name: Vetting DTS run: go vet -v ./... From ba9373e1232f3934051d534aa8b8861a24b66417 Mon Sep 17 00:00:00 2001 From: "Jeffrey N. Johnson" Date: Wed, 3 Jun 2026 10:44:14 -0700 Subject: [PATCH 2/3] Disabling KBase auth2 server tests (disallowed by Cloudflare bot storm security) --- auth/kbase_auth_server_test.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/auth/kbase_auth_server_test.go b/auth/kbase_auth_server_test.go index f122b242..eef88d5c 100644 --- a/auth/kbase_auth_server_test.go +++ b/auth/kbase_auth_server_test.go @@ -138,6 +138,7 @@ func getKbaseToken() (string, bool) { func TestNewKBaseAuthServer(t *testing.T) { assert := assert.New(t) + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") devToken, ok := getKbaseToken() if ok { // this test requires a valid developer token @@ -160,6 +161,7 @@ func TestNewKBaseAuthServer(t *testing.T) { func TestInvalidToken(t *testing.T) { assert := assert.New(t) + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") _, ok := getKbaseToken() if ok { // test against the real server @@ -188,6 +190,7 @@ func TestNoIdentifiers(t *testing.T) { } // test with the mock server + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") server, err := NewKBaseAuthServer("no_idents_token", func(cfg *KBaseAuthServerConfig) { cfg.BaseURL = mockKBaseServer.URL @@ -206,6 +209,7 @@ func TestNoOrcID(t *testing.T) { } // test with the mock server + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") server, err := NewKBaseAuthServer("no_orcid_token", func(cfg *KBaseAuthServerConfig) { cfg.BaseURL = mockKBaseServer.URL @@ -219,6 +223,7 @@ func TestNoOrcID(t *testing.T) { func TestClient(t *testing.T) { assert := assert.New(t) + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") devToken, ok := getKbaseToken() if ok { // this test requires a valid developer token with an associated ORCID From 94aeb5ae91430287a4bb328701ef1b42eea54f2c Mon Sep 17 00:00:00 2001 From: "Jeffrey N. Johnson" Date: Wed, 3 Jun 2026 10:54:35 -0700 Subject: [PATCH 3/3] Moving t.Skip statements upward. --- auth/kbase_auth_server_test.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/auth/kbase_auth_server_test.go b/auth/kbase_auth_server_test.go index eef88d5c..bf7efb1e 100644 --- a/auth/kbase_auth_server_test.go +++ b/auth/kbase_auth_server_test.go @@ -136,9 +136,9 @@ func getKbaseToken() (string, bool) { // tests whether a proxy for the KBase authentication server can be // constructed func TestNewKBaseAuthServer(t *testing.T) { + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") assert := assert.New(t) - t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") devToken, ok := getKbaseToken() if ok { // this test requires a valid developer token @@ -159,9 +159,9 @@ func TestNewKBaseAuthServer(t *testing.T) { // tests whether an invalid KBase token prevents a proxy for the auth server // from being constructed func TestInvalidToken(t *testing.T) { + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") assert := assert.New(t) - t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") _, ok := getKbaseToken() if ok { // test against the real server @@ -182,6 +182,7 @@ func TestInvalidToken(t *testing.T) { // tests that the proxy handles missing identifiers correctly func TestNoIdentifiers(t *testing.T) { + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") assert := assert.New(t) _, ok := getKbaseToken() @@ -190,7 +191,6 @@ func TestNoIdentifiers(t *testing.T) { } // test with the mock server - t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") server, err := NewKBaseAuthServer("no_idents_token", func(cfg *KBaseAuthServerConfig) { cfg.BaseURL = mockKBaseServer.URL @@ -201,6 +201,7 @@ func TestNoIdentifiers(t *testing.T) { // tests that the proxy handles missing OrcID identifiers correctly func TestNoOrcID(t *testing.T) { + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") assert := assert.New(t) _, ok := getKbaseToken() @@ -209,7 +210,6 @@ func TestNoOrcID(t *testing.T) { } // test with the mock server - t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") server, err := NewKBaseAuthServer("no_orcid_token", func(cfg *KBaseAuthServerConfig) { cfg.BaseURL = mockKBaseServer.URL @@ -221,9 +221,9 @@ func TestNoOrcID(t *testing.T) { // tests whether the authentication server can return information for the // client (the user associated with the specified developer token) func TestClient(t *testing.T) { + t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") assert := assert.New(t) - t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)") devToken, ok := getKbaseToken() if ok { // this test requires a valid developer token with an associated ORCID