nestingReduce: invert if cond, replace body with `continue`, move old body after the statement (gocritic)

unlambda: replace `func(c *cli.Context) error {

`marshalling` is a misspelling of `marshaling` (misspell)

exported: type name will be used as validation.ValidationResponse by other packages, and that stutters; consider calling this Response (revive)

exported: type name will be used as validation.ValidationRequest by other packages, and that stutters; consider calling this Request (revive)

superfluous-else: if block ends with call to panic function, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)

superfluous-else: if block ends with call to panic function, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)

unused-parameter: parameter 'req' seems to be unused, consider removing or renaming it as _ (revive)

var-naming: var errHttp should be errHTTP (revive)

var-declaration: should omit type [5]time.Duration from declaration of var timeouts; it will be inferred from the right-hand side (revive)

challenge_test.go:1407: Error Trace: /home/runner/work/pki-acme-service/pki-acme-service/acme/challenge_test.go:1407 Error: Received unexpected error: unexpected challenge type "" Test: TestChallenge_Validate/fail/already-invalid ---

wire_integration_test.go:137: database file name: /tmp/integration-db-3788444679 wire_integration_test.go:190: directory: {https://test.ca.smallstep.com/acme/test@acme-%3Ctest%3Eprovisioner.com/new-nonce https://test.ca.smallstep.com/acme/test@acme-%3Ctest%3Eprovisioner.com/new-account https://test.ca.smallstep.com/acme/test@acme-%3Ctest%3Eprovisioner.com/new-order https://test.ca.smallstep.com/acme/test@acme-%3Ctest%3Eprovisioner.com/revoke-cert https://test.ca.smallstep.com/acme/test@acme-%3Ctest%3Eprovisioner.com/key-change <nil>} wire_integration_test.go:203: nonce: aWR3ZzlHUUFoMWVyVzE2SVJMVENSeFdGbnViZnVGdHo wire_integration_test.go:237: account ID: fJwZZQCKNzjiYY7wOpjLlwHqhw25ffIN wire_integration_test.go:278: authzs IDs: [i0KNJXKTkb0SODLDjFzgj1nUROuala6Y aPogQXyJgP9DSB9fu2YPGUIw6OM5UBKm] time="2024-11-04T15:11:25Z" level=warning msg="marking challenge fBpFBNxg1Crh2XnFGYJCC2e7fkRBIzWv as invalid: keyAuthorization does not match; expected \"ROiS9C3Vxkl2UE56JW8AfBYXzwvNLx9O.vY6niH6azF3uPb-DkwhKzTaSqtE3eBo7_XGsWNCD7Ag\", but got \"token.vY6niH6azF3uPb-DkwhKzTaSqtE3eBo7_XGsWNCD7Ag\"" wire_integration_test.go:428: challenge: fBpFBNxg1Crh2XnFGYJCC2e7fkRBIzWv invalid time="2024-11-04T15:11:25Z" level=warning msg="marking challenge 2QsiYt1hnq7aLcR9kqcxhEz2gF1B6vsg as invalid: failed validating Wire access token: failed validation: go-jose/go-jose/jwt: validation failed, invalid audience claim (aud)" wire_integration_test.go:428: challenge: 2QsiYt1hnq7aLcR9kqcxhEz2gF1B6vsg invalid wire_integration_test.go:452: updated challenge: fBpFBNxg1Crh2XnFGYJCC2e7fkRBIzWv valid wire_integration_test.go:452: updated challenge: 2QsiYt1hnq7aLcR9kqcxhEz2gF1B6vsg valid wire_integration_test.go:490: updated order status: ready wire_integration_test.go:580: Error Trace: /home/runner/work/pki-acme-service/pki-acme-service/acme/api/wire_integration_test.go:580 /home/runner/work/pki-acme-service/pki-acme-service/acme/api/wire_integration_test.go:586 Error: Not equal: expected: "valid" actual : "processing" Diff: --- Expected +++ Actual @@ -1,2 +1,2 @@ -(acme.Status) (len=5) "valid" +(acme.Status) (len=10) "processing" Test: TestWireIntegration ---

Process completed with exit code 2.

The job was canceled because "stable" failed.

The operation was canceled.