[database] Add request to optionalDependencies

[styfle]

The database package uses the request dependency but doesn't include it in package.json.

See this line that calls require('request'):

firebase-js-sdk/packages/database/src/nodePatches.ts

Line 162 in 6b53e00

(FirebaseIFrameScriptHolder as any).request = /** @type {function({url: string, forever: boolean}, function(Error, number, string))} */ require('request');

This adds request to the optionalDependencies (perhaps it should be under dependencies?)

I did not create an issue because it's easier to discuss with the code in this PR 😄

[styfle]

Would you prefer "2.x" instead of asterisk?

[mikelehen]

Thanks for the PR! Your change looks reasonable, but I'm discussing with a colleague and I think we may opt to just delete this code. It's not really used at this point. Hang tight.

[styfle]

@mikelehen That sounds good too, thanks!

Feel free to close this PR if you need to.

[mikelehen]

Moving forward with the delete: #2144

So I'll close this but we do appreciate you brining this to our attention. 😀