The port option should be either number or string type

[mattdeluco]

Issue Description

The port option should accept both strings and numbers. The NodeJS URL
API defines port as a string, where this value ultimately ends up.

It should not be necessary to parseInt(process.env.PORT), for example.

What are you doing?

I am trying to create an instance of Sequelize, configuring the port for the connection using an environment variable process.env.PORT.

import { Sequelize } from 'sequelize';

const seq = new Sequelize({
  port: process.env.PORT || '5432'
});

What do you expect to happen?

I wanted to use environment variables for the port option

What is actually happening?

I am having to use parseInt() to convert a string to a number

No overload matches this call.
  Overload 1 of 4, '(options?: Options): Sequelize', gave the following error.
    Type 'string' is not assignable to type 'number'.
  Overload 2 of 4, '(uri: string, options?: Options): Sequelize', gave the following error.
    Argument of type '{ port: string; }' is not assignable to parameter of type 'string'.ts(2769)
sequelize.d.ts(239, 3): The expected type comes from property 'port' which is declared here on type 'Options'

Additional context

Environment

• Sequelize version: 4063c2a
• Node.js version: v14.5.0
• Operating System: macOS 11.1
• If TypeScript related: TypeScript version: XXX

Issue Template Checklist

How does this problem relate to dialects?

• I think this problem happens regardless of the dialect.
• I think this problem happens only for the following dialect(s):
• I don't know, I was using PUT-YOUR-DIALECT-HERE, with connector library version XXX and database version XXX

Would you be willing to resolve this issue by submitting a Pull Request?

I have created draft pull request #13024, I do not have time to carry it further.

• Yes, I have the time and I know how to start.
• Yes, I have the time but I don't know how to start, I would need guidance.
• No, I don't have the time, although I believe I could do it if I had the time...
• No, I don't have the time and I wouldn't even know how to start.

[github-actions]

This issue has been automatically marked as stale because it has been open for 7 days without activity. It will be closed if no further activity occurs. If this is still an issue, just leave a comment or remove the "stale" label. 🙂

[arman-amiri]

port: Number(process.env?.DB_PORT),