Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
Trac
Trac
  • Project overview
    • Project overview
    • Details
    • Activity
  • Issues 246
    • Issues 246
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Operations
    • Operations
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Create a new issue
  • Issue Boards
Collapse sidebar

GitLab is used only for code review, issue tracking and project management. Canonical locations for source code are still https://gitweb.torproject.org/ https://git.torproject.org/ and git-rw.torproject.org.

  • Legacy
  • TracTrac
  • Issues
  • #7687

Closed
Open
Opened Dec 09, 2012 by Trac@tracbot

RFC: add close_stream method to Controller

Damian,

I have finally settled on what I think is a reasonable approach to StreamClosureReason (SCR) and RelayEndReason (RER) enums. But I am still working on tests for close_stream, so RFC for now.

  1. Does my handling of the *Reason enums work for you? My research in the tor source and torspec indicates that the reasons the tor process sends in the STREAM event are direct mappings of the reason the relay ended. Except, with STREAM (FAILED|CLOSED|DETACHED), REASON=END and the "real" reason is in REMOTE_REASON.

  2. How do you feel about adding the SocksiPy http://sourceforge.net/projects/socksipy/ module under test/ ? This is a BSD-licensed module that I hope will make stream-using integ tests possible. But I have not yet tried, so maybe it won't help.

This work is in the exp-close-stream-v1 branch on git://gitorious.org/stem-robinson/stem-robinson.git and commit logs can be read at https://gitorious.org/stem-robinson/stem-robinson/commits/exp-close-stream-v1

Trac:
Username: robinson

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
Reference: legacy/trac#7687