Expand port type to accept a string

[corydeppen]

I've run into an issue deploying a GraphQL server on Azure. Since iisnode is used to run Node apps, the port is assigned to a named pipe instead of a number (see tjanczuk/iisnode#46). As a result, I see an error RangeError: "port" argument must be >= 0 and < 65536 when the server tries to run.

I'm hoping you'll find it reasonable to modify the port definition in the Options interface to be number | string and remove parseInt from the conditional when assigning the port option in the GraphQLServer class.

I'm happy to submit a PR if you agree.

[schickling]

Thanks a lot for bringing this up @corydeppen. Just to clarify: When running the server via IIS under Windows the port will become a string?

[corydeppen]

Correct, at least when the iisnode handler is used like in the case of Azure.

[schickling]

Sounds good! Yeah, would be great to have a PR for this.

[abduncan]

I just ran into this! Glad to see its fixed already :D