Opened 10 years ago
Closed 10 years ago
#6028 enhancement closed wontfix (wontfix)
Documentation for request encoders points to wrong interfaces
Reported by: | therve | Owned by: | therve |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | web | Keywords: | documentation easy |
Cc: | jknight | Branch: |
branches/fix-encoder-doc-6028
branch-diff, diff-cov, branch-cov, buildbot |
Author: | therve |
Description
When merging #104 I kept wrong links to interfaces when renaming them, it should be fxied. OTOH I'm not sure pointing to private interfaces is really useful.
Change History (6)
comment:1 Changed 10 years ago by
Cc: | jknight added |
---|
comment:2 Changed 10 years ago by
Author: | → therve |
---|---|
Branch: | → branches/fix-encoder-doc-6028 |
comment:3 Changed 10 years ago by
Keywords: | easy review added |
---|
i removed the references as it seems more clear to me.
comment:4 Changed 10 years ago by
Keywords: | review removed |
---|---|
Owner: | set to therve |
Fix seems correct as far as it goes.
However I have a concern about #104, related to the SSL/zlib CRIME attack. I wonder if the encoders
argument to Site
is safe at all (if it isn't, then maybe it's not worth applying documentation fixes).
comment:5 Changed 10 years ago by
therve offered to introduce a more focusable API for encoders before the next Twisted release, or revert the original merge to avoid committing us to supporting it. That seems fine to me. So, go ahead and merge this if you like (and please file a ticket for that adjusted API). Thanks!
comment:6 Changed 10 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
I'll fix the documentation in the new ticket, #6031.
(In [35843]) Branching to 'fix-encoder-doc-6028'