Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate ContextFactory (and friends) #7215

Open
twisted-trac opened this issue Apr 19, 2014 · 0 comments
Open

Deprecate ContextFactory (and friends) #7215

twisted-trac opened this issue Apr 19, 2014 · 0 comments

Comments

@twisted-trac
Copy link

glyph's avatar @glyph reported
Trac ID trac#7215
Type task
Created 2014-04-19 20:26:03Z

twisted.internet.ssl.ContextFactory is a badly-designed implicit interface which nobody should implement themselves. Nobody should have implemented it before either, but after #7098 lands there are actual documented interfaces you can use to live dangerously.

This is also something that should be removed sooner in its deprecation life-cycle than most things. ContextFactory's insistence that applications should provide their own special verification logic using OpenSSL's minefield of an API is potentially actively harmful to security, so we should get rid of it and force applications written under the old, bad regime to reconsider one of the nice new APIs that have since been added.

It may be good to have a server-side equivalent high-level interface to the client-side one added in #7098 before we pull the trigger on that though.

Searchable metadata
trac-id__7215 7215
type__task task
reporter__glyph glyph
priority__normal normal
milestone__ 
branch__ 
branch_author__ 
status__new new
resolution__None None
component__core core
keywords__ 
time__1397939163000000 1397939163000000
changetime__1397939163000000 1397939163000000
version__None None
owner__ 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant