Make sure that the CircleCI API token is truly required.

Review Request #12482 — Created July 21, 2022 and submitted

Information

rbintegrations
release-3.x

Reviewers

When configuring CircleCI, the API token field is required, but this is
only an aspect of the form field. We ended up running into some
situations (particularly in unit testing) where the integration was
configured but the API token was unset. In this case, we'd end up with
bad requests.

This change checks for an empty or missing API token, and preemptively
fails the operation. The error will be logged as well as included in the
status update.

  • Tested that attempting to start a request with a missing API token
    produced the correct log and error.
  • Ran unit tests.
Summary ID
Make sure that the CircleCI API token is truly required.
When configuring CircleCI, the API token field is required, but this is only an aspect of the form field. We ended up running into some situations (particularly in unit testing) where the integration was configured but the API token was unset. In this case, we'd end up with bad requests. This change checks for an empty or missing API token, and preemptively fails the operation. The error will be logged as well as included in the status update. Testing Done: - Tested that attempting to start a request with a missing API token produced the correct log and error. - Ran unit tests.
72ee7e53f9aaa71a6ab3083c5d3d61153ac1c33a
chipx86
  1. Ship It!
  2. 
      
maubin
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.x (a18be5b)
Loading...