Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • W websockettest
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Terraform modules
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • foss
  • websockettest
  • Merge requests
  • !9

Ensure we only await once

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Peter Farr requested to merge peter/only-await-once into master Feb 08, 2022
  • Overview 0
  • Commits 1
  • Pipelines 0
  • Changes 3

In practice returning the PendingRequest is clunky because we have to await both the send() call, and the returned PendingRequest. This instead awaits the PendingRequest and returns the result directly, which results in a much nicer API for sending websocket requests.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: peter/only-await-once