Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

suggestion: getAvailablePort() #3279

Closed
harrysolovay opened this issue Mar 24, 2023 · 7 comments · Fixed by #3890
Closed

suggestion: getAvailablePort() #3279

harrysolovay opened this issue Mar 24, 2023 · 7 comments · Fixed by #3890
Labels
help wanted Extra attention is needed

Comments

@harrysolovay
Copy link
Contributor

harrysolovay commented Mar 24, 2023

Are any of the following utilities good candidates for inclusion?

withSignal

Run an async function by passing in a fresh abort controller's signal. Upon completion, abort.

Use case: ensuring that the resolution of a promise triggers the abort, which may trigger other things.

async function withSignal<T>(cb: (signal: AbortSignal) => Promise<T>) {
  const controller = new AbortController()
  try {
    return await cb(controller.signal)
  } finally {
    controller.abort()
  }
}

port.getAvailable

Determine an open port.

Use case: get an unused port when launching processes that allow for specificity of port.

function getAvailable(): number {
  const tmp = Deno.listen({ port: 0 })
  const { port } = tmp.addr as Deno.NetAddr
  tmp.close()
  return port
}

port.ready

Poll whether a given port can be connected to

Use case: determine whether a server is prepared to accept connections.

async function ready(port: number, ms = 500): Promise<void> {
  while (true) {
    try {
      const connection = await Deno.connect({ port })
      connection.close()
      break
    } catch (e) {
      if (e instanceof Deno.errors.ConnectionRefused) await delay(ms)
      else throw e
    }
  }
}
@jsejcksn
Copy link
Contributor

Although use cases are obvious if one has engaged in the related topics, I think an explainer is useful for each (and also that this is probably better-suited for a discussion topic than an issue).

@harrysolovay
Copy link
Contributor Author

I've just updated the description with use cases.

Regarding issue vs. discussion: it seems I'm unable to convert this issue into a discussion. Would someone with the necessary permissions please do so.

@kt3k
Copy link
Member

kt3k commented Nov 28, 2023

I like getAvailable and ready ideas. I guess the module name might be std/net?

I don't see the use case of withSignal very well. Do you have any use case in mind?

@iuioiua
Copy link
Contributor

iuioiua commented Nov 28, 2023

I'm of a similar opinion. Yeah, I can't think of a use case for withSignal() right now.

@iuioiua iuioiua changed the title misc. ideas suggestion: port.getAvailable() and port.ready() Nov 30, 2023
@iuioiua iuioiua added the help wanted Extra attention is needed label Nov 30, 2023
@iuioiua
Copy link
Contributor

iuioiua commented Nov 30, 2023

PRs are welcome.

@iuioiua
Copy link
Contributor

iuioiua commented Dec 1, 2023

On second thought, I don't think these functions are the best approach.

Instead of using port.getAvailable(), one could just use port = 0, which automatically chooses a free port. Are there any use cases that defy that?

And instead of using port.ready() one could just use:

import { retry } from "https://deno.land/[email protected]/async/retry.ts";
await retry(async () => await Deno.connect({ port: 8000 }));

@harrysolovay
Copy link
Contributor Author

It's useful if someone needs to specify the port for other parts of a program before actually opening the connection. Good point re using retry.

@iuioiua iuioiua changed the title suggestion: port.getAvailable() and port.ready() suggestion: port.getAvailable() Dec 3, 2023
@iuioiua iuioiua changed the title suggestion: port.getAvailable() suggestion: getAvailablePort() Dec 3, 2023
@kt3k kt3k closed this as completed in #3890 Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants