[Bug 58497] Unable to have a custom implementation of AbstractHttp11Processor due to package private methods

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=58497

Mark Thomas  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #3 from Mark Thomas  ---
Tomcat's connectors were originally designed with this sort of use in mind
although it is likely that over time some changes have crept in that are
unhelpful in this regard.

Ideally, you should be able to do this by extending existing abstract base
classes / overriding existing implementations depending on how close the
current code is to what you need. If you spot changes (splitting up methods,
changing visibility, adding accessors, etc.) that would make this easier we are
happy to consider them. (Please open additional BZ issue for such requests.)

I have made the methods in question protected in 8.0.x and 7.0.x.

The connectors were refactored further in 8.0.x and significantly so in 9.0.x.
It is worth you taking a look at 9.0.x as  that is still in development and any
changes you might need so enable a future upgrade to 9.0.x will be much easier
to make sooner rather than later.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 58497] Unable to have a custom implementation of AbstractHttp11Processor due to package private methods

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=58497

--- Comment #4 from Amit Pande  ---
Thanks a lot, Mark, all for your quick responses and invaluable suggestions.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 58497] Unable to have a custom implementation of AbstractHttp11Processor due to package private methods

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=58497

Amit Pande  changed:

   What|Removed |Added

 Status|NEEDINFO|NEW

--- Comment #2 from Amit Pande  ---
Apologies Mark & Violeta for a delayed reply. Trying to explain below what we
need off tomcat.


"Requirement is to have multiple custom Acceptor strategy and at least each
Acceptor strategy is taken up by one thread.

One and possibly many acceptor thread could be listening for incoming
HTTP/S request.
One acceptor thread would be accepting request from legacy JNI which in
turn registers with a custom service handlers to accept incoming requests.

In  the current framework this is not possible or at least I could not
figure out how to do it.

So easiest way I found was to copy paste default connector code and change
acceptor strategy.
Now I have two connectors, default one listening for HTTP/S request.
Custom connector acceptor thread relying on JNI library for handing over
to it new connection from custom service handler. "



Would really appreciate if you could point to appropriate way to solve this
issue.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 58497] Unable to have a custom implementation of AbstractHttp11Processor due to package private methods

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=58497

Violeta Georgieva  changed:

   What|Removed |Added

 Status|NEW |NEEDINFO

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 58497] Unable to have a custom implementation of AbstractHttp11Processor due to package private methods

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=58497

Amit Pande  changed:

   What|Removed |Added

   Priority|P2  |P1
   Severity|critical|blocker

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 58497] Unable to have a custom implementation of AbstractHttp11Processor due to package private methods

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=58497

--- Comment #1 from Mark Thomas  ---
That class is not considered part of Tomcat's public API (see the README file).
It is liable to change between any point release.

What is the underlying requirement that means you need to extend that class?

Note that in 9.0.x this class no longer exists and it is likely the same
changes will be back-ported to 8.x in the near future.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 58497] Unable to have a custom implementation of AbstractHttp11Processor due to package private methods

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=58497

Amit Pande  changed:

   What|Removed |Added

 CC||amit_pa...@symantec.com

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org