Review comments

Author: rix0rrr

packages/aws-cdk/lib/api/aws-auth/sdk-provider.ts

@@ -301,7 +301,6 @@ function parseHttpOptions(options: SdkHttpOptions) {
     debug('Using proxy server: %s', proxyAddress);
     // eslint-disable-next-line @typescript-eslint/no-require-imports
     const ProxyAgent: any = require('proxy-agent');
-    // tslint:disable-next-line:no-console
     config.httpOptions.agent = new ProxyAgent(proxyAddress);
   }
   if (caBundlePath) {

packages/aws-cdk/test/api/sdk-provider.test.ts

@@ -154,11 +154,14 @@ describe('CLI compatible credentials loading', () => {
     // Messy mocking
     let called = false;
     jest.mock('proxy-agent', () => {
+      // eslint-disable-next-line @typescript-eslint/no-require-imports
       class FakeAgent extends require('https').Agent {
         public addRequest(_: any, __: any) {
-          // This error takes 6 seconds to be completely handled. It might be retries in the SDK
-          // somewhere, or something about the Node event loop. I've spent an hour trying to figure
-          // it out and I can't, and I gave up. We'll just have to live with this.
+          // FIXME: this error takes 6 seconds to be completely handled. It
+          // might be retries in the SDK somewhere, or something about the Node
+          // event loop. I've spent an hour trying to figure it out and I can't,
+          // and I gave up. We'll just have to live with this until someone gets
+          // inspired.
           const error = new Error('ABORTED BY TEST');
           (error as any).code = 'RequestAbortedError';
           (error as any).retryable = false;