Jetty

Improve client API

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 6.1.5
  • Component/s: None
  • Labels:
    None
  • Testcase included:
    yes
  • Number of attachments :
    3
  1. jetty-client.patch
    01/Mar/07 1:57 PM
    47 kB
    Guillaume Nodet
  2. jetty-client.patch
    27/Feb/07 3:19 PM
    41 kB
    Guillaume Nodet
  3. jetty-core.patch
    27/Feb/07 3:17 PM
    3 kB
    Guillaume Nodet

Issue Links

Activity

Hide
Guillaume Nodet added a comment -

Patch to the NIO layer to support client connections

Show
Guillaume Nodet added a comment - Patch to the NIO layer to support client connections
Hide
Guillaume Nodet added a comment -

Patch to the client API.
The client API works

Show
Guillaume Nodet added a comment - Patch to the client API. The client API works
Hide
Guillaume Nodet added a comment -

Updated patch to support sockets / blocking / select endpoints

Show
Guillaume Nodet added a comment - Updated patch to support sockets / blocking / select endpoints
Hide
Greg Wilkins added a comment -

Guillaume - awesome!

Totally in the right direction!

Do you want to apply on codehaus xircles for contributor status and then you can have direct access to the
jetty-contrib svn. You would still need to run patches to jetty-core through me!

I have not gone totally over your code yet, but have seen a few little items of concern... but I will
double check before I expand on them. I have not used the concurrent mechanism much before and
am cautious about things done outside of big safe synchronize blocks

I will also start work on a proxy handler that uses this, which will be a good test of it.

Anyway, patches have been committed. THANKS A MILLION!

Show
Greg Wilkins added a comment - Guillaume - awesome! Totally in the right direction! Do you want to apply on codehaus xircles for contributor status and then you can have direct access to the jetty-contrib svn. You would still need to run patches to jetty-core through me! I have not gone totally over your code yet, but have seen a few little items of concern... but I will double check before I expand on them. I have not used the concurrent mechanism much before and am cautious about things done outside of big safe synchronize blocks I will also start work on a proxy handler that uses this, which will be a good test of it. Anyway, patches have been committed. THANKS A MILLION!
Hide
Greg Wilkins added a comment -

patched long ago

Show
Greg Wilkins added a comment - patched long ago

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: