Ticket #4176 defect closed fixed

Opened 3 years ago

Last modified 3 years ago

twistd web "-n" option documentation is backwards

Reported by: exarkun Owned by:
Priority: normal Milestone:
Component: web Keywords: documentation, easy
Cc: jesstess Branch:
Author: Launchpad Bug:

Description

twistd web --help produces, amongst other things, this output: 

  -n, --notracebacks      Display tracebacks in broken web pages. Displaying
                          tracebacks to users may be security risk!

Okay... the option is called notracebacks and the behavior of supplying it is for tracebacks to be displayed? Not likely. Displaying tracebacks is the default, and this option really does disable that.

Attachments

notracebacks.patch Download (3.0 KB) - added by jesstess 3 years ago.

Change History

1

Changed 3 years ago by jesstess

  • cc jesstess added

Grepping around I see:

  • twisted/python/zsh/_mktap:'(--notracebacks)-n[Display tracebacks in broken web pages. Displaying tracebacks to users may be security risk!]' \
  • twisted/python/zsh/_mktap:'(-n)--notracebacks[Display tracebacks in broken web pages. Displaying tracebacks to users may be security risk!]' \
  • twisted/python/zsh/_twistd:'(--notracebacks)-n[Display tracebacks in broken web pages. Displaying tracebacks to users may be security risk!]' \
  • twisted/python/zsh/_twistd:'(-n)--notracebacks[Display tracebacks in broken web pages. Displaying tracebacks to users may be security risk!]' \
  • twisted/web/tap.py: ["notracebacks", "n", "Display tracebacks in broken web pages. " +

So these all probably need changing.

2

Changed 3 years ago by jesstess

  • owner changed from jknight to jesstess

Changed 3 years ago by jesstess

3

Changed 3 years ago by jesstess

  • owner jesstess deleted
  • keywords easy, review added; easy removed

Display tracebacks ==> Do not display tracebacks, for the 5 cases above.

4

Changed 3 years ago by TimAllen

I've reviewed this and I approve of it being committed, but I can't actually commit it myself. I'll leave this ticket in the review queue, waiting for somebody with commit access.

5

Changed 3 years ago by jesstess

  • owner set to jesstess
  • keywords easy added; easy, review removed

6

Changed 3 years ago by jesstess

  • status changed from new to closed
  • resolution set to fixed

(In [27854]) Apply notracebacks.patch

Author: jesstess Reviewer: TimAllen Fixes: #4176

twistd web '-n' documentation should say that it _disables_ tracebacks.

7

Changed 2 years ago by <automation>

  • owner jesstess deleted
Note: See TracTickets for help on using tickets.