Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • S slapos.core
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 39
    • Merge requests 39
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • slapos.core
  • Merge requests
  • !343
You need to sign in or sign up before continuing.

testing: fix resource warning with ManagedHTTPServer

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Jérome Perrin requested to merge fix/testing-resourcewarnings into master Nov 05, 2021
  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 2

We fork a subprocess to serve requests, but the listening socket is bound in server constructor and inherited by subprocess. With this pattern, we have to explicitly close the socket from parent process to prevent the leak.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/testing-resourcewarnings
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7