Fix request import in log.ts

[silasbw]

Use the TypeScript CommonJS-style import:

https://www.typescriptlang.org/docs/handbook/modules.html

because it looks like the request types use the CommonJS-style export:

https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/request/index.d.ts#L397

CC #274

[brendandburns]

/lgtm
/approve

It's weird that our CI/CD didn't catch this...

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brendandburns, silasbw

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [brendandburns]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[drubin]

@brendanburns @silasbw any way we could make CI catch this type of error?

I feel like we can't rely on remembering all these different style imports to always get it right. I know that both @brendandburns and I sometimes refactor (in the past) the import style to be consistent.

[silasbw]

i had a repro locally. it was specific to log.ts. i will spend a minute to see if i can convert that to a more holistic/inclusive CI test.