[GitHub] activemq-artemis pull request #617: ARTEMIS-607 Added interceptor support fo...

2016-07-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/617 --- 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

[GitHub] activemq-artemis issue #617: ARTEMIS-607 Added interceptor support for MQTT ...

2016-07-05 Thread johnament
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/617 Ok, I rebased from master and squashed. --- 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

[GitHub] activemq-artemis pull request #622: ARTEMIS-578 cert authn/z for STOMP

2016-07-05 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/622#discussion_r69647599 --- Diff: artemis-protocols/artemis-stomp-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/stomp/v11/StompFrameHandlerV11.java ---

[GitHub] activemq-artemis pull request #622: ARTEMIS-578 cert authn/z for STOMP

2016-07-05 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/622#discussion_r69647099 --- Diff: artemis-protocols/artemis-stomp-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/stomp/v11/StompFrameHandlerV11.java

[GitHub] activemq-artemis pull request #622: ARTEMIS-578 cert authn/z for STOMP

2016-07-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/622 ARTEMIS-578 cert authn/z for STOMP You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-578

[GitHub] activemq-artemis pull request #620: ARTEMIS-612 Improving Failback's max rep...

2016-07-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/620 --- 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

[GitHub] activemq-artemis pull request #621: ARTEMIS-584 add validated user to msg

2016-07-05 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/621#discussion_r69644037 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/spi/core/security/ActiveMQJAASSecurityManager.java --- @@ -80,31 +81,41

[GitHub] activemq-artemis pull request #621: ARTEMIS-584 add validated user to msg

2016-07-05 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/621#discussion_r69641957 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/spi/core/security/ActiveMQJAASSecurityManager.java --- @@ -80,31 +81,41

[GitHub] activemq-artemis pull request #621: ARTEMIS-584 add validated user to msg

2016-07-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/621 ARTEMIS-584 add validated user to msg Implements a new feature to aid in security auditing by adding the name of the validated user to the messages it sends. You can merge this pull

[GitHub] activemq-artemis issue #621: ARTEMIS-584 add validated user to msg

2016-07-05 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/621 I ran the full test-suite. There were no new failures. --- 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

[GitHub] activemq-artemis pull request #610: Artemis build enhancements

2016-07-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/610 --- 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

[GitHub] activemq-artemis pull request #620: ARTEMIS-612 Improving Failback's max rep...

2016-07-05 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/620 ARTEMIS-612 Improving Failback's max replication The server will always restart now, with older files being removed. The system will now move current data into ./oldreplica.#,

[GitHub] activemq-artemis pull request #610: Artemis build enhancements

2016-07-05 Thread glauciom
Github user glauciom commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/610#discussion_r69613608 --- Diff: pom.xml --- @@ -112,8 +112,8 @@ UTF-8 UTF-8 - 1.7 - 1.7 + 1.8 + 1.8

[GitHub] activemq-artemis pull request #616: Various cleanups

2016-07-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/616 --- 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

[GitHub] activemq-artemis issue #617: ARTEMIS-607 Added interceptor support for MQTT ...

2016-07-05 Thread johnament
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/617 Up to you. I won't be able to squash until this evening. --- 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

[GitHub] activemq-artemis pull request #613: ARTEMIS-602 Fix REST examples

2016-07-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/613 --- 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

[GitHub] activemq-artemis pull request #619: ARTEMIS-610 update clientFailureCheckPer...

2016-07-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/619 --- 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

[GitHub] activemq-artemis pull request #610: Artemis build enhancements

2016-07-05 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/610#discussion_r69594043 --- Diff: pom.xml --- @@ -112,8 +112,8 @@ UTF-8 UTF-8 - 1.7 - 1.7 + 1.8 +

[GitHub] activemq-artemis issue #617: ARTEMIS-607 Added interceptor support for MQTT ...

2016-07-05 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/617 These look good.. I can squash them at the time of the merge. I will run the whole testsuite just in case. (It takes 2 hours or so now) --- If your project is set up

[GitHub] activemq-artemis pull request #613: ARTEMIS-602 Fix REST examples

2016-07-05 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/613#discussion_r69583184 --- Diff: examples/features/standard/rest/dup-send/README.txt --- @@ -1,5 +1,5 @@ System Requirements: -You will need JDK 1.6 and

Time for yet another board report, please lend a hand

2016-07-05 Thread Bruce Snyder
Hi All, I hope everyone in the northern hemisphere is enjoying their summer. It is that time again, please take a few moments to help compile the development activity within the project over the last three months: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=65145183 Your

[GitHub] activemq-artemis pull request #618: Upgrade ApacheRat to 0.12

2016-07-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/618 --- 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

[GitHub] activemq-artemis issue #617: ARTEMIS-607 Added interceptor support for MQTT ...

2016-07-05 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/617 Could you squash these two commits together? --- 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

Re: RAT Checks get stuck on Artemis Integration Test Module

2016-07-05 Thread Robbie Gemmell
Changing to the -n option to -f gives a preview listing of what would be deleted. You can use -e flags to prevent deleting certain things, like any IDE config dirs. On 5 July 2016 at 00:26, John D. Ament wrote: > Doing a git clean fixed it. But last time I blindly run a