Opened 4 years ago

Closed 19 months ago

#8251 closed enhancement (implemented)

Port arm's tor config utils

Reported by: atagar Owned by: atagar
Priority: Low Milestone:
Component: Core Tor/Stem Version:
Severity: Normal Keywords: controller
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Arm has tor configuration utilities for both reading the man page contents and validating the user's torrc. We should port these to stem and add unit/integ tests.

Child Tickets

Change History (7)

comment:1 Changed 4 years ago by atagar

Related ticket: #7654

comment:2 Changed 4 years ago by gsathya

I'm sorry, but why does a tor controller library need to read man page contents?

comment:3 follow-up: Changed 4 years ago by atagar

I'm sorry, but why does a tor controller library need to read man page contents?

To be able to provide a description of tor's config options. See arm's editor panel for an example.

comment:4 in reply to: ↑ 3 Changed 4 years ago by gsathya

Replying to atagar:

To be able to provide a description of tor's config options. See arm's editor panel for an example.

Sounds like an arm specific use case. I don't understand where anyone would want a "controller" library to parse the man page.

comment:5 follow-up: Changed 4 years ago by atagar

Sounds like an arm specific use case. I don't understand where anyone would want a "controller" library to parse the man page.

You're focusing too much on how it works rather than what it does. 'Give me a human readable description of config option X' is a nice feature to have. And no, this isn't arm specific. Vidalia and other GUI controllers, for instance, would love to have this feature.

comment:6 in reply to: ↑ 5 Changed 4 years ago by gsathya

Replying to atagar:

Sounds like an arm specific use case. I don't understand where anyone would want a "controller" library to parse the man page.

You're focusing too much on how it works rather than what it does. 'Give me a human readable description of config option X' is a nice feature to have. And no, this isn't arm specific. Vidalia and other GUI controllers, for instance, would love to have this feature.

Exactly, the GUI's should implement this if they need it not stem. It just sounded like feature creep. Anyhow, it's your call :)

comment:7 Changed 19 months ago by atagar

  • Resolution set to implemented
  • Severity set to Normal
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.