Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-82330121
Can you close this one please ? I have imported your commit into mine
---
If your project is set up for it, you can reply to this email and have your
reply
Github user mastanik closed the pull request at:
https://github.com/apache/activemq-6/pull/182
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user mastanik commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-81538505
Yes, it's ok.
If there would be any problems with my PR please let me know.
---
If your project is set up for it, you can reply to this email and have your
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-81980131
as I promised I would work on this after the release...#191
The release is now under vote, so I continued your work at #191 where I did
some
Github user mastanik commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-81176956
Added 2 new commits with outgoing frames support and new stomp interceptor
paragrap
---
If your project is set up for it, you can reply to this email and have your
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-81177534
I will merge it later today when I get home
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-81347599
There's a risk of something wrong with your PR. We have a release tomorrow
so I will wait until the release is done before I merge this, ok?
I'm
Github user HornetQBot commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-78891438
Can one of the admins verify this patch by saying ok to test?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user mastanik opened a pull request:
https://github.com/apache/activemq-6/pull/182
Added possibility to intercept stomp frames
Initially I was trying to log all the incoming stomp frames from the client
on a server side, but could not find any easy way of doing this. This pr
Github user mastanik commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-78994222
Yes, sure, I will update this pr when it is done
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-79020069
I will hold the PR open until you do it then
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-79020971
If you have a chance also, add a paragraph to the docs :) I will do it
right away if you don't have time to do it.. but please advise if that's the
case so I
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-78967583
Nicely done... +1 on me for a merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user mastanik commented on the pull request:
https://github.com/apache/activemq-6/pull/182#issuecomment-79029355
Sure, I will update it as well
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
14 matches
Mail list logo