[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-17 Thread clebertsuconic
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-17 Thread mastanik
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-16 Thread mastanik
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-16 Thread clebertsuconic
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-15 Thread mastanik
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-15 Thread clebertsuconic
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-15 Thread clebertsuconic
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread HornetQBot
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread mastanik
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread mastanik
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread clebertsuconic
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread clebertsuconic
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread clebertsuconic
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] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread mastanik
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