chore(deps): bump golang.org/x/oauth2 from 0.25.0 to 0.28.0 (#531)

Author: Gustavo Bazan

.github/workflows/pr.yml

@@ -19,7 +19,7 @@ jobs:
       - name: lint
         uses: golangci/[email protected]
         with:
-          version: v1.57.1
+          version: v1.64.7
           args: --timeout=10m
       - name: tests
         run: make test
@@ -48,4 +48,3 @@ jobs:
       - name: Fail when generate introducing diff
         run: |
           git diff --exit-code
-          

.golangci.yml

@@ -18,6 +18,7 @@ linters-settings:
       - nestingReduce
       - sloppyReassign
       - typeAssertChain
+      - builtinShadow
   gocyclo:
     min-complexity: 15
   revive:
@@ -54,37 +55,17 @@ linters-settings:
       # We still want empty methods for consistency - do not control if method will be empty.
       # - name: unused-receiver
       - name: unreachable-code
-      - name: redefines-builtin-id
-  tagliatelle:
-    # check the struck tag name case
-    case:
-      use-field-name: true
-      rules:
-        json: camel
-        yaml: camel
-
 linters:
   # please, do not use `enable-all`: it's deprecated and will be removed soon.
   # inverted configuration with `enable-all` and `disable` is not scalable during updates of golangci-lint
   disable-all: true
   enable:
     - bodyclose # checks whether HTTP response body is closed successfully [fast: false, auto-fix: false]
-    #- depguard # Go linter that checks if package imports are in a list of acceptable packages [fast: false, auto-fix: false]
     - dogsled # Checks assignments with too many blank identifiers (e.g. x, _, _, _, := f()) [fast: true, auto-fix: false]
     - errcheck # Errcheck is a program for checking for unchecked errors in go programs. These unchecked errors can be critical bugs in some cases [fast: false, auto-fix: false]
     - errorlint # errorlint is a linter for that can be used to find code that will cause problems with the error wrapping scheme introduced in Go 1.13. [fast: false, auto-fix: false]
-    # Upstream code intentionally have number of non exhaustive checks
-    # - exhaustive # check exhaustiveness of enum switch statements [fast: false, auto-fix: false]
-    - exportloopref # checks for pointers to enclosing loop variables [fast: false, auto-fix: false]
     - gochecknoinits # Checks that no init functions are present in Go code [fast: true, auto-fix: false]
     - gocritic # Provides many diagnostics that check for bugs, performance and style issues. [fast: false, auto-fix: false]
-    # Needs to be disabled for generated content as complexity comes from OpenAPI
-    # - gocyclo # Computes and checks the cyclomatic complexity of functions [fast: true, auto-fix: false]
-    # We do not control comment content - it is OpenAPI driven
-    # - godot # Check if comments end in a period [fast: true, auto-fix: true]
-    # HTML format of our comments break this check thinking that files aren't formatted
-    # - gofmt # Gofmt checks whether code was gofmt-ed. By default this tool runs with -s option to check for code simplification [fast: true, auto-fix: true]
-    # - goimports # Goimports does everything that gofmt does. Additionally it checks unused imports [fast: true, auto-fix: true]
     - goprintffuncname # Checks that printf-like functions are named with `f` at the end [fast: true, auto-fix: false]
     - gosec # Inspects source code for security problems [fast: false, auto-fix: false]
     - gosimple # Linter for Go source code that specializes in simplifying a code [fast: false, auto-fix: false]
@@ -100,53 +81,3 @@ linters:
     - unconvert # Remove unnecessary type conversions [fast: false, auto-fix: false]
     - unparam # Reports unused function parameters [fast: false, auto-fix: false]
     - unused # Checks Go code for unused constants, variables, functions and types [fast: false, auto-fix: false]
-    # Disable as it is hard to avoid whitespace when building templates
-    # - whitespace # Tool for detection of leading and trailing whitespace [fast: true, auto-fix: true]
-
-  # don't enable:
-#    Replaced by revive
-#    - stylecheck # Stylecheck is a replacement for golint [fast: false, auto-fix: false]
-#    - ineffassign # Detects when assignments to existing variables are not used [fast: true, auto-fix: false]
-#    - asciicheck # Simple linter to check that your code does not contain non-ASCII identifiers [fast: true, auto-fix: false]
-#    - cyclop # checks function and package cyclomatic complexity [fast: false, auto-fix: false]
-#    - deadcode # Finds unused code [fast: false, auto-fix: false]
-#    - dupl # Tool for code clone detection [fast: true, auto-fix: false]
-#    - durationcheck # check for two durations multiplied together [fast: false, auto-fix: false]
-#    - exhaustivestruct # Checks if all struct's fields are initialized [fast: false, auto-fix: false]
-#    - forbidigo # Forbids identifiers [fast: true, auto-fix: false]
-#    - forcetypeassert # finds forced type assertions [fast: true, auto-fix: false]
-#    - funlen # Tool for detection of long functions [fast: true, auto-fix: false]
-#    - gci # Gci control golang package import order and make it always deterministic. [fast: true, auto-fix: true]
-#    - gochecknoglobals # check that no global variables exist [fast: true, auto-fix: false]
-#    - gocognit # Computes and checks the cognitive complexity of functions [fast: true, auto-fix: false]
-#    - goconst # Finds repeated strings that could be replaced by a constant [fast: true, auto-fix: false]
-#    - godox # Tool for detection of FIXME, TODO and other comment keywords [fast: true, auto-fix: false]
-#    - gomnd # An analyzer to detect magic numbers. [fast: true, auto-fix: false]
-#    - goerr113 # Golang linter to check the errors handling expressions [fast: false, auto-fix: false]
-#    - gofumpt # Gofumpt checks whether code was gofumpt-ed. [fast: true, auto-fix: true]
-#    - goheader # Checks is file header matches to pattern [fast: true, auto-fix: false]
-#    - gomoddirectives # Manage the use of 'replace', 'retract', and 'excludes' directives in go.mod. [fast: true, auto-fix: false]
-#    - gomodguard # Allow and block list linter for direct Go module dependencies. This is different from depguard where there are different block types for example version constraints and module recommendations. [fast: true, auto-fix: false]
-#    - importas # Enforces consistent import aliases [fast: false, auto-fix: false]
-#    - interfacer # Linter that suggests narrower interface types [fast: false, auto-fix: false]
-#    - lll # Reports long lines [fast: true, auto-fix: false]
-#    - makezero # Finds slice declarations with non-zero initial length [fast: false, auto-fix: false]
-#    - maligned # Tool to detect Go structs that would take less memory if their fields were sorted [fast: false, auto-fix: false]
-#    - nestif # Reports deeply nested if statements [fast: true, auto-fix: false]
-#    - nilerr # Finds the code that returns nil even if it checks that the error is not nil. [fast: false, auto-fix: false]
-#    - nlreturn # nlreturn checks for a new line before return and branch statements to increase code clarity [fast: true, auto-fix: false]
-#    - noctx # noctx finds sending http request without context.Context [fast: false, auto-fix: false]
-#    - paralleltest # paralleltest detects missing usage of t.Parallel() method in your Go test [fast: true, auto-fix: false]
-#    - prealloc # Finds slice declarations that could potentially be preallocated [fast: true, auto-fix: false]
-#    - predeclared # find code that shadows one of Go's predeclared identifiers [fast: true, auto-fix: false]
-#    - promlinter # Check Prometheus metrics naming via promlint [fast: true, auto-fix: false]
-#    - scopelint # Scopelint checks for unpinned variables in go programs [fast: true, auto-fix: false]
-#    - sqlclosecheck # Checks that sql.Rows and sql.Stmt are closed. [fast: false, auto-fix: false]
-#    - structcheck # Finds unused struct fields [fast: false, auto-fix: false]
-#    - testpackage # linter that makes you use a separate _test package [fast: true, auto-fix: false]
-#    - tparallel # tparallel detects inappropriate usage of t.Parallel() method in your Go test codes [fast: false, auto-fix: false]
-#    - wastedassign # wastedassign finds wasted assignment statements. [fast: false, auto-fix: false]
-#    - wrapcheck # Checks that errors returned from external packages are wrapped [fast: false, auto-fix: false]
-#    - wsl # Whitespace Linter - Forces you to use empty lines! [fast: true, auto-fix: false]
-#    - varcheck # Finds unused global variables and constants [fast: false, auto-fix: false]
-#    - ifshort # Checks that your code uses short syntax for if-statements whenever possible [fast: true, auto-fix: false]

Makefile

@@ -1,5 +1,5 @@
 SOURCE_FILES?=./...
-GOLANGCI_VERSION=v1.59.0
+GOLANGCI_VERSION=v1.64.7
 GOIMPORTS_VERSION=v0.21.0
 COVERAGE=coverage.out
 

examples/go.mod

@@ -1,8 +1,8 @@
 module go.mongodb.org/atlas-sdk/v20250219001/examples
 
-go 1.22.0
+go 1.23.0
 
-toolchain go1.23.1
+toolchain go1.24.1
 
 replace go.mongodb.org/atlas-sdk/v20250219001 => ../
 
@@ -12,13 +12,12 @@ require (
 	go.mongodb.org/atlas-sdk/v20250219001 v20250219001.1.0
 )
 
-require golang.org/x/oauth2 v0.25.0 // indirect
-
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/hashicorp/go-cleanhttp v0.5.2 // indirect
 	github.com/pmezard/go-difflib v1.0.0 // indirect
 	github.com/stretchr/objx v0.5.2 // indirect
 	github.com/stretchr/testify v1.10.0
+	golang.org/x/oauth2 v0.28.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

examples/go.sum

@@ -22,8 +22,8 @@ github.com/stretchr/objx v0.5.2 h1:xuMeJ0Sdp5ZMRXx/aWO6RZxdr3beISkG5/G/aIRr3pY=
 github.com/stretchr/objx v0.5.2/go.mod h1:FRsXN1f5AsAjCGJKqEizvkpNtU+EGNCLh3NxZ/8L+MA=
 github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
 github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
-golang.org/x/oauth2 v0.25.0 h1:CY4y7XT9v0cRI9oupztF8AgiIu99L/ksR/Xp/6jrZ70=
-golang.org/x/oauth2 v0.25.0/go.mod h1:XYTD2NtWslqkgxebSiOHnXEap4TF09sJSc7H1sXbhtI=
+golang.org/x/oauth2 v0.28.0 h1:CrgCKl8PPAVtLnU3c+EDw6x11699EWlsDeWNWKdIOkc=
+golang.org/x/oauth2 v0.28.0/go.mod h1:onh5ek6nERTohokkhCD/y2cV4Do3fxFHFuAejCkRWT8=
 golang.org/x/sys v0.20.0 h1:Od9JTbYCk261bKm4M/mw7AklTlFYIa0bIp9BgSm1S8Y=
 golang.org/x/sys v0.20.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=

go.mod

@@ -1,17 +1,19 @@
 module go.mongodb.org/atlas-sdk/v20250219001
 
-go 1.22.0
+go 1.23.0
+
+toolchain go1.23.7
 
 require (
 	github.com/go-test/deep v1.1.1
 	github.com/mongodb-forks/digest v1.1.0
-	golang.org/x/oauth2 v0.25.0
+	github.com/stretchr/testify v1.10.0
+	golang.org/x/oauth2 v0.28.0
 )
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/pmezard/go-difflib v1.0.0 // indirect
 	github.com/stretchr/objx v0.5.2 // indirect
-	github.com/stretchr/testify v1.10.0
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

go.sum

@@ -12,8 +12,8 @@ github.com/stretchr/objx v0.5.2 h1:xuMeJ0Sdp5ZMRXx/aWO6RZxdr3beISkG5/G/aIRr3pY=
 github.com/stretchr/objx v0.5.2/go.mod h1:FRsXN1f5AsAjCGJKqEizvkpNtU+EGNCLh3NxZ/8L+MA=
 github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
 github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
-golang.org/x/oauth2 v0.25.0 h1:CY4y7XT9v0cRI9oupztF8AgiIu99L/ksR/Xp/6jrZ70=
-golang.org/x/oauth2 v0.25.0/go.mod h1:XYTD2NtWslqkgxebSiOHnXEap4TF09sJSc7H1sXbhtI=
+golang.org/x/oauth2 v0.28.0 h1:CrgCKl8PPAVtLnU3c+EDw6x11699EWlsDeWNWKdIOkc=
+golang.org/x/oauth2 v0.28.0/go.mod h1:onh5ek6nERTohokkhCD/y2cV4Do3fxFHFuAejCkRWT8=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=