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

HTTP::Daemon::ClientConn should be decoupled from HTTP::Daemon [rt.cpan.org #7466] #8

Open
oalders opened this issue Apr 1, 2019 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@oalders
Copy link
Member

oalders commented Apr 1, 2019

Migrated from rt.cpan.org#7466 (status was 'new')

Requestors:

From [email protected] on 2004-08-25 02:33:47
:

I guess this is best classified as a "Wishlist item."

HTTP::Daemon::ClientConn should be decoupled from HTTP::Daemon, which 
would allow users to use HTTP::Daemon::ClientConn (which contains the 
bulk of the code) with other daemon modules (such as Net::Daemon, 
Net::Server, etc.).

As it is now, you can pull this off by subclassing HTTP::Daemon and 
re-implementing a bunch of its methods, but that seems unnecessarily 
messy. (I took this approach when using HTTP::Daemon to build a proxy 
server along with Net::Server.)

It appears that HTTP::Daemon::ClientConn could be tweaked to reduce or 
eliminate the coupling.

I'd be happy to donate the code to do this, if there is interest. (I 
don't want to do this on speculation, as I don't think it is significant 
enough to warrant forking the module, and in my experience, module 
maintainers are usually reluctant to accept patches of this nature if 
they don't fit their vision of how the module should be architected.)

  -Tom



@genio genio added enhancement New feature or request help wanted Extra attention is needed labels Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants