fix: remove logger

Author: BobHemphill76

src/keys/get-management-token.spec.ts

@@ -10,7 +10,7 @@ import {
  GetManagementTokenOptions,
 } from './get-management-token'
 import { HttpError } from '../utils'
-import { Logger } from '../utils'
+// import { Logger } from '../utils'
 import { sign } from 'jsonwebtoken'
 import { LRUCache } from 'lru-cache'
 
@@ -24,7 +24,7 @@ const DEFAULT_OPTIONS: GetManagementTokenOptions = {
  environmentId: ENVIRONMENT_ID,
  reuseToken: false,
 }
-const noop = () => {}
+// const noop = () => {}
 
 const defaultCache: LRUCache<string, string> = new LRUCache({ max: 10 })
 let fetchStub: sinon.SinonStub
@@ -40,10 +40,10 @@ afterEach(() => {
 describe('getManagementToken', () => {
  it('fetches a token', async () => {
  const mockToken = 'token'
- const logger = noop as unknown as Logger
+ // const logger = noop as unknown as Logger
  fetchStub.resolves({ ok: true, status: 201, json: () => Promise.resolve({ token: mockToken }) })
  const getManagementToken = createGetManagementToken(
- logger,
+ // logger,
  { prefixUrl: '', retry: { limit: 0 } },
  defaultCache,
  )
@@ -62,14 +62,14 @@ describe('getManagementToken', () => {
  })
 
  it('caches token while valid', async () => {
- const logger = noop as unknown as Logger
+ // const logger = noop as unknown as Logger
  const mockToken = sign({ a: 'b' }, 'a-secret-key', {
  expiresIn: '10 minutes',
  })
 
  fetchStub.resolves({ ok: true, status: 201, json: () => Promise.resolve({ token: mockToken }) })
  const getManagementToken = createGetManagementToken(
- logger,
+ // logger,
  { prefixUrl: '', retry: { limit: 0 } },
  defaultCache,
  )
@@ -84,15 +84,15 @@ describe('getManagementToken', () => {
  })
 
  it('does not cache expired token', async () => {
- const logger = noop as unknown as Logger
+ // const logger = noop as unknown as Logger
  const mockToken = sign({ a: 'b' }, 'a-secret-key', {
  expiresIn: '5 minutes',
  })
 
  fetchStub.resolves({ ok: true, status: 201, json: () => Promise.resolve({ token: mockToken }) })
  const cache: LRUCache<string, string> = new LRUCache({ max: 10 })
  const getManagementToken = createGetManagementToken(
- logger,
+ // logger,
  { prefixUrl: '', retry: { limit: 0 } },
  cache,
  )
@@ -118,14 +118,14 @@ describe('getManagementToken', () => {
  describe('when using a keyId', () => {
  it('fetches a token', async () => {
  const mockToken = 'token'
- const logger = noop as unknown as Logger
+ // const logger = noop as unknown as Logger
  fetchStub.resolves({
  ok: true,
  status: 201,
  json: () => Promise.resolve({ token: mockToken }),
  })
  const getManagementToken = createGetManagementToken(
- logger,
+ // logger,
  { prefixUrl: '', retry: { limit: 0 } },
  defaultCache,
  )
@@ -158,10 +158,10 @@ describe('getManagementToken', () => {
 
  describe('when having API problems', () => {
  it(`throws when API returns an error`, async () => {
- const logger = noop as unknown as Logger
+ // const logger = noop as unknown as Logger
  fetchStub.rejects(new HttpError({ statusCode: 500 } as unknown as Response))
  const getManagementToken = createGetManagementToken(
- logger,
+ // logger,
  { prefixUrl: '', retry: { limit: 0 } },
  defaultCache,
  )

src/keys/get-management-token.ts

@@ -1,7 +1,7 @@
 import * as jwtImpl from 'jsonwebtoken'
 import type { SignOptions } from 'jsonwebtoken'
 import { LRUCache } from 'lru-cache'
-import { createLogger, Logger, createValidateStatusCode } from '../utils'
+import { createValidateStatusCode } from '../utils'
 import { FetchOptions, makeRequest, withDefaultOptions, withHook, withRetry } from '../utils/http'
 
 const jwt = 'default' in jwtImpl ? jwtImpl.default : jwtImpl
@@ -24,18 +24,19 @@ let defaultCache: LRUCache<string, string> | undefined
 const generateOneTimeToken = (
  privateKey: string,
  { appId, keyId }: { appId: string; keyId?: string },
- { log }: { log: Logger },
+ // { log }: { log: Logger },
 ): string => {
- log('Signing a JWT token with private key')
+ // log('Signing a JWT token with private key')
+ // eslint-disable-next-line no-useless-catch
  try {
  const baseSignOptions: SignOptions = { algorithm: 'RS256', issuer: appId, expiresIn: '10m' }
  const signOptions: SignOptions = keyId ? { ...baseSignOptions, keyid: keyId } : baseSignOptions
 
  const token = sign({}, privateKey, signOptions)
- log('Successfully signed token')
+ // log('Successfully signed token')
  return token
  } catch (e) {
- log('Unable to sign token')
+ // log('Unable to sign token')
  throw e
  }
 }
@@ -47,11 +48,11 @@ const getTokenFromOneTimeToken = async (
  spaceId,
  environmentId,
  }: { appInstallationId: string; spaceId: string; environmentId: string },
- { log, fetchOptions }: { log: Logger; fetchOptions: FetchOptions },
+ { /* log, */ fetchOptions }: { /* log: Logger; */ fetchOptions: FetchOptions },
 ): Promise<string> => {
  const validateStatusCode = createValidateStatusCode([201])
 
- log(`Requesting CMA Token with given App Token`)
+ // log(`Requesting CMA Token with given App Token`)
 
  const requestor = makeRequest(
  `/spaces/${spaceId}/environments/${environmentId}/app_installations/${appInstallationId}/access_tokens`,
@@ -67,9 +68,9 @@ const getTokenFromOneTimeToken = async (
  const retryRequestor = withRetry(hookRequestor, fetchOptions)
  const response = await retryRequestor()
 
- log(
- `Successfully retrieved CMA Token for app ${appInstallationId} in space ${spaceId} and environment ${environmentId}`,
- )
+ // log(
+ //  `Successfully retrieved CMA Token for app ${appInstallationId} in space ${spaceId} and environment ${environmentId}`,
+ // )
 
  return ((await response.json()) as { token: string }).token
 }
@@ -79,7 +80,7 @@ const getTokenFromOneTimeToken = async (
  * @internal
  */
 export const createGetManagementToken = (
- log: Logger,
+ // log: Logger,
  fetchOptions: FetchOptions,
  cache: LRUCache<string, string>,
 ) => {
@@ -104,9 +105,9 @@ export const createGetManagementToken = (
  const appToken = generateOneTimeToken(
  privateKey,
  { appId: opts.appInstallationId, keyId: opts.keyId },
- { log },
+ // { log },
  )
- const ott = await getTokenFromOneTimeToken(appToken, opts, { log, fetchOptions })
+ const ott = await getTokenFromOneTimeToken(appToken, opts, { /* log, */ fetchOptions })
  if (opts.reuseToken) {
  const decoded = decode(ott)
  if (decoded && typeof decoded === 'object' && decoded.exp) {
@@ -151,7 +152,7 @@ export const getManagementToken = async (privateKey: string, opts: GetManagement
  const httpClientOpts = typeof opts.host !== 'undefined' ? { prefixUrl: opts.host } : {}
 
  return createGetManagementToken(
- createLogger({ filename: __filename }),
+ // createLogger({ filename: __filename }),
  withDefaultOptions(httpClientOpts),
  defaultCache!,
  )(privateKey, opts)