Ticket #5870 enhancement new

Opened 9 months ago

Last modified 9 months ago

Make a string parser interface for client endpoints that takes the reactor as a parameter in its signature

Reported by: ashfall Owned by: ashfall
Priority: normal Milestone:
Component: core Keywords: endpoint
Cc: Branch:
Author: Launchpad Bug:

Description

At present, IStreamClientEndpointStringParser.parseStreamClient doesn't take the reactor as a parameter, which is bad. As a result, even though endpoints.clientFromString needs a reactor to be passed as a parameter, it cannot pass it on to string description plugins for client endpoints, and we end up having to use a global reactor in the plugin, which is bad too.

We need a new interface with the correct signature defined, a new plugin interface to go with it, the plugin loader updated to load that new kind of plugin and call its method correctly, deprecation warnings for use of the old interface, and the existing plugins/parser in Twisted to be ported to the new interface.

Can/should probably be split into smaller tickets.

Change History

1

Changed 9 months ago by ashfall

  • summary changed from Make a string parser interface for client endpoints that takes the reactor as a parameter in it's signature to Make a string parser interface for client endpoints that takes the reactor as a parameter in its signature
Note: See TracTickets for help on using tickets.