Net::Stomp-0.31 Patch to Support Bytes Message

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Net::Stomp-0.31 Patch to Support Bytes Message

sileshi
I have added support for "content-length" header field to handle
Bytes Messages from Java JMS or Perl Stom clients.
1. Net::Stomp.pm
   - Rewrote the receive_frame() method to read the header first,
     and based on the "content-length" if any, read the body.
   - Applied binmode to socket to avoid problems on Windows

2. Net::Stomp::Frame
    - Rewrote parse() method to fix problems I run into.
      this also makes it robust using the power of regular expression.

I have attached the two files below:

 Stomp.pm

 Frame.pm
Reply | Threaded
Open this post in threaded view
|

Re: Net::Stomp-0.31 Patch to Support Bytes Message

chirino
Did you ever review the patch that I attached to:
http://issues.apache.org/activemq/browse/AMQ-978


On 10/23/06, sileshi <[hidden email]> wrote:

>
>
> I have added support for "content-length" header field to handle
> Bytes Messages from Java JMS or Perl Stom clients.
> 1. Net::Stomp.pm
>    - Rewrote the receive_frame() method to read the header first,
>      and based on the "content-length" if any, read the body.
>    - Applied binmode to socket to avoid problems on Windows
>
> 2. Net::Stomp::Frame
>     - Rewrote parse() method to fix problems I run into.
>       this also makes it robust using the power of regular expression.
>
> I have attached the two files below:
>
>   http://www.nabble.com/file/3810/Stomp.pm Stomp.pm
>
>   http://www.nabble.com/file/3811/Frame.pm Frame.pm
> --
> View this message in context:
> http://www.nabble.com/Net%3A%3AStomp-0.31-Patch-to-Support-Bytes-Message-tf2496525.html#a6959274
> Sent from the ActiveMQ - User mailing list archive at Nabble.com.
>
>


--
Regards,
Hiram

Blog: http://hiramchirino.com
Reply | Threaded
Open this post in threaded view
|

Re: Net::Stomp-0.31 Patch to Support Bytes Message

sileshi
I just reviewed it and it looks fine. I have also updated the AMQ-978 and made
available my patch. I will let Leon choose the patch.

I appreciate your help in this regard.

On another subject: As you suggested, I checked out
4.1 snapshot and attempt to build it maven 2.0.4, but I keep having
build error on POM. here is the error I'm getting(let me know if I miss
something):

[ERROR] BUILD ERROR
[INFO] ------------------------------------------------------------------------
[INFO] Error building POM (may not be this project's POM).


Project ID: directory-protocols:kerberos-protocol

Reason: Error getting POM for 'directory-protocols:kerberos-protocol' from the repository: Error transferring file
  directory-protocols:kerberos-protocol:pom:0.5

from the specified remote repositories:
  central (http://repo1.maven.org/maven2),
  apache.snapshots (http://people.apache.org/repo/m2-snapshot-repository),
  apache.incubating.releases (http://people.apache.org/repo/m2-incubating-repository)



[INFO] ------------------------------------------------------------------------
[INFO] For more information, run Maven with the -e switch
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 9 minutes 31 seconds
[INFO] Finished at: Mon Oct 23 15:10:43 CDT 2006
[INFO] Final Memory: 7M/13M
[INFO] ------------------------------------------------------------------------

Hiram Chirino wrote
Did you ever review the patch that I attached to:
http://issues.apache.org/activemq/browse/AMQ-978


On 10/23/06, sileshi <sileshikassa@yahoo.com> wrote:
>
>
> I have added support for "content-length" header field to handle
> Bytes Messages from Java JMS or Perl Stom clients.
> 1. Net::Stomp.pm
>    - Rewrote the receive_frame() method to read the header first,
>      and based on the "content-length" if any, read the body.
>    - Applied binmode to socket to avoid problems on Windows
>
> 2. Net::Stomp::Frame
>     - Rewrote parse() method to fix problems I run into.
>       this also makes it robust using the power of regular expression.
>
> I have attached the two files below:
>
>   http://www.nabble.com/file/3810/Stomp.pm Stomp.pm
>
>   http://www.nabble.com/file/3811/Frame.pm Frame.pm
> --
> View this message in context:
> http://www.nabble.com/Net%3A%3AStomp-0.31-Patch-to-Support-Bytes-Message-tf2496525.html#a6959274
> Sent from the ActiveMQ - User mailing list archive at Nabble.com.
>
>


--
Regards,
Hiram

Blog: http://hiramchirino.com