right, thats great information thanks. I opened up the port I just need to test it out now. But there is not a specific ip addr I should be listening on with my server? Should my current code work?
Binding to INADDR_ANY just means to listen on all available interfaces no matter what their address. As you're likely to have just one (real, apart from the loopback) network interface that's just what you need. The port forwarding on the router takes care of the rest.