--allow-net=:8080 broken

[brandonkal]

If any code uses shorthand notation:

import { serve } from "https://deno.land/std/http/server.ts"
const PORT = 8080
serve(`:${PORT}`)

then any variant of

deno run --allow-net=:8080 server.ts
deno run --allow-net=localhost:8080 server.ts
deno run --allow-net=0.0.0.0:8080 server.ts
deno run --allow-net=127.0.0.1:8080 server.ts

fails.

Unfortunately, a lot of libraries in x use this pattern which makes locking down permissions impossible without forking all those libraries.

[ry]

Maybe this was introduced in 74c2605 ?

I think this shouldn't be a terrible difficult or involved fix. I will mark as "Good First Issue".

[KrisChambers]

I would like to take a stab at this one.

[jacobgc]

Just tested this with deno version 1.1.2 which is before my commit (74c2605) and can confirm that this issue is still present.

[utkzas]

I also want to contribute to this project (denojs). Can someone tell me how should I start?

[jsouto18]

I've tested this on the latest release and I've managed to execute the code successfully running deno run --allow-net=:8080 server.ts or deno run --allow-net=0.0.0.0:8080 server.ts which seems to be the intended behaviour as the serve function binds to 0.0.0.0:8080.
@ry is this the intended behaviour?
The only other thing I found odd was trying to run serve(127.0.0.1:8080) and running deno run --allow-net=localhost:8080 server.ts fails because localhost is not resolved to the configured IP, for example in /etc/hosts.

[bartlomieju]

Fixed in #6698