Re: [PR] AMQ-9765: Upgrade to Derby 10.17.1.0 [activemq]

2025-09-13 Thread via GitHub
jbonofre closed pull request #1489: AMQ-9765: Upgrade to Derby 10.17.1.0 URL: https://github.com/apache/activemq/pull/1489 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To un

Re: [PR] [AMQ-9768] Fix JMX value escaping in web console. [activemq]

2025-09-13 Thread via GitHub
jbonofre commented on PR #1487: URL: https://github.com/apache/activemq/pull/1487#issuecomment-3288855955 I would suggest to split into two PRs: 1. One just fixing the formatting that we can also apply on 6.1.x 2. One changing the interface (adding name) that will be apply just on `main

[PR] AMQ-9763: Upgrade to commons-io 2.20.0 [activemq]

2025-09-13 Thread via GitHub
jbonofre opened a new pull request, #1491: URL: https://github.com/apache/activemq/pull/1491 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

[PR] AMQ-9764: Upgrade to Camel 4.14.0 [activemq]

2025-09-13 Thread via GitHub
jbonofre opened a new pull request, #1490: URL: https://github.com/apache/activemq/pull/1490 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-13 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2344550412 ## docs/user-manual/proxy-protocol.adoc: ## @@ -0,0 +1,59 @@ += PROXY Protocol +:idprefix: +:idseparator: - +:docinfo: shared + +As noted in the official htt

Re: [PR] [AMQ-9768] Fix JMX value escaping in web console. [activemq]

2025-09-13 Thread via GitHub
thezbyg commented on PR #1487: URL: https://github.com/apache/activemq/pull/1487#issuecomment-3287647778 The main issue is missing JMX value escaping. It could be fixed without any broker interface changes by adding JMXSupport.encodeObjectNamePart() calls in BrokerFacadeSupport.java file, b

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2345901239 ## tests/e2e-tests/src/test/java/org/apache/activemq/artemis/tests/e2e/proxy_protocol/HAProxyTest.java: ## @@ -0,0 +1,334 @@ +/* + * Licensed to the Apache So

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2345899586 ## docs/user-manual/proxy-protocol.adoc: ## @@ -0,0 +1,59 @@ += PROXY Protocol +:idprefix: +:idseparator: - +:docinfo: shared + +As noted in the official htt

Re: [PR] NO JIRA - typo fix [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor merged PR #116: URL: https://github.com/apache/activemq-artemis-console/pull/116 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsub

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
tabish121 commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2345462152 ## tests/e2e-tests/src/test/java/org/apache/activemq/artemis/tests/e2e/proxy_protocol/HAProxyTest.java: ## @@ -0,0 +1,334 @@ +/* + * Licensed to the Apache S

[PR] AMQ-9765: Upgrade to Derby 10.17.1.0 [activemq]

2025-09-12 Thread via GitHub
jbonofre opened a new pull request, #1489: URL: https://github.com/apache/activemq/pull/1489 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

[PR] AMQ-9766: Upgrade to Jetty 11.0.26 [activemq]

2025-09-12 Thread via GitHub
jbonofre opened a new pull request, #1488: URL: https://github.com/apache/activemq/pull/1488 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

Re: [PR] AMQ-9767: Upgrade to jmdns 3.6.2 [activemq]

2025-09-12 Thread via GitHub
jbonofre merged PR #1486: URL: https://github.com/apache/activemq/pull/1486 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@activemq.

Re: [PR] ARTEMIS-5647 - Fix branding in webpack dev server [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor merged PR #115: URL: https://github.com/apache/activemq-artemis-console/pull/115 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsub

Re: [PR] ARTEMIS-5630 - Fix webpack hot reload [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor merged PR #119: URL: https://github.com/apache/activemq-artemis-console/pull/119 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsub

Re: [PR] [AMQ-9768] Fix JMX value escaping in web console. [activemq]

2025-09-12 Thread via GitHub
jbonofre commented on PR #1487: URL: https://github.com/apache/activemq/pull/1487#issuecomment-3286667949 If I understand the need here, it looks like an improvement more than a bug fix as it changes some "interfaces" (introducing name field). -- This is an automated message from the Apac

Re: [PR] ARTEMIS-5648 - Remove deprecated HPKP http header [activemq-artemis-console]

2025-09-12 Thread via GitHub
grgrzybek commented on PR #117: URL: https://github.com/apache/activemq-artemis-console/pull/117#issuecomment-3285077203 It happens ;) I've created hawtio/hawtio#4070 to remove tests and the filter itself -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] ARTEMIS-5647 - Fix branding in webpack dev server [activemq-artemis-console]

2025-09-12 Thread via GitHub
GChuf commented on PR #115: URL: https://github.com/apache/activemq-artemis-console/pull/115#issuecomment-3284518742 Rebased! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2345239933 ## tests/e2e-tests/src/test/java/org/apache/activemq/artemis/tests/e2e/proxy_protocol/HAProxyTest.java: ## @@ -0,0 +1,334 @@ +/* + * Licensed to the Apache So

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2344483955 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -215,7 +215,7 @@ protected void decode(ChannelHandlerCon

Re: [PR] ARTEMIS-4924 handle invalid messages in SNF queue [activemq-artemis]

2025-09-12 Thread via GitHub
gtully commented on PR #5921: URL: https://github.com/apache/activemq-artemis/pull/5921#issuecomment-3284518462 My only question, if a message ends up on the SNF DLQ or on the default DLQ, is it now usable from there? Do we need to strip the message of the SNF routing context? Ideally

Re: [PR] ARTEMIS-5656 Prevent federation from creating self sustaining loops [activemq-artemis]

2025-09-12 Thread via GitHub
tabish121 commented on PR #5920: URL: https://github.com/apache/activemq-artemis/pull/5920#issuecomment-3285452581 Think I've addresses all the typos -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[PR] ARTEMIS-5658 - fix reloading of filter from storage [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor opened a new pull request, #121: URL: https://github.com/apache/activemq-artemis-console/pull/121 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] [AMQ-9768] Fix JMX value escaping in web console. [activemq]

2025-09-12 Thread via GitHub
thezbyg opened a new pull request, #1487: URL: https://github.com/apache/activemq/pull/1487 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-m

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2344590083 ## artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/TransportConstants.java: ## @@ -525,13 +530,13 @@ private static int

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2344578744 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/HAProxyMessageHandler.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the A

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-12 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2344571228 ## tests/e2e-tests/pom.xml: ## @@ -331,6 +362,62 @@ + +maven-resources-plugin + +

[PR] ARTEMIS-5548 - fix adding back the retry for dlq and expiry [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor opened a new pull request, #122: URL: https://github.com/apache/activemq-artemis-console/pull/122 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] ARTEMIS-5656 Prevent federation from creating self sustaining loops [activemq-artemis]

2025-09-12 Thread via GitHub
gemmellr merged PR #5920: URL: https://github.com/apache/activemq-artemis/pull/5920 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@a

Re: [PR] [AMQ-9749] Update JaasDualAuthenticationBroker isSSL method to be pro… [activemq]

2025-09-12 Thread via GitHub
mattrpav merged PR #1476: URL: https://github.com/apache/activemq/pull/1476 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@activemq.

Re: [PR] ARTEMIS-5658 - fix reloading of filter from storage [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor commented on PR #121: URL: https://github.com/apache/activemq-artemis-console/pull/121#issuecomment-3285452926 @GChuf can you review please, I think we lost this functionality with your recent filter changes -- This is an automated message from the Apache Git Service. To respo

Re: [PR] ARTEMIS-5657 - Invoke all HTTP header filters in loop [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor merged PR #120: URL: https://github.com/apache/activemq-artemis-console/pull/120 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsub

Re: [PR] AMQ-9743: Fixes Web Console column sorting. [activemq]

2025-09-12 Thread via GitHub
jbonofre merged PR #1479: URL: https://github.com/apache/activemq/pull/1479 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@activemq.

Re: [PR] ARTEMIS-5648 - Remove deprecated HPKP http header [activemq-artemis-console]

2025-09-12 Thread via GitHub
GChuf commented on PR #117: URL: https://github.com/apache/activemq-artemis-console/pull/117#issuecomment-3285042694 I saw that, but only after creating this PR :) The idea then, was to make a PR on hawtio first, and then use the io.hawt.web.filters.ResponseHeadersFilter in artemis conso

[PR] ARTEMIS-5630 - Fix webpack hot reload [activemq-artemis-console]

2025-09-12 Thread via GitHub
GChuf opened a new pull request, #119: URL: https://github.com/apache/activemq-artemis-console/pull/119 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsu

Re: [PR] ARTEMIS-5631 - Update hawtio to v4.5 [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor merged PR #114: URL: https://github.com/apache/activemq-artemis-console/pull/114 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsub

Re: [PR] ARTEMIS-5656 Prevent federation from creating self sustaining loops [activemq-artemis]

2025-09-12 Thread via GitHub
gemmellr commented on code in PR #5920: URL: https://github.com/apache/activemq-artemis/pull/5920#discussion_r2343541349 ## tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/connect/AMQPFederationServerToServerTest.java: ## @@ -1923,4 +1924

Re: [PR] ARTEMIS-5648 - Remove deprecated HPKP http header [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor merged PR #117: URL: https://github.com/apache/activemq-artemis-console/pull/117 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsub

Re: [PR] ARTEMIS-5631 - Update hawtio to v4.5 [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor commented on PR #114: URL: https://github.com/apache/activemq-artemis-console/pull/114#issuecomment-3284392252 tested npm run and all looks good. will merge -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] ARTEMIS-5424 - Allow listing all items in tables [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor merged PR #99: URL: https://github.com/apache/activemq-artemis-console/pull/99 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubsc

Re: [PR] ARTEMIS-5424 - Allow listing all items in tables [activemq-artemis-console]

2025-09-12 Thread via GitHub
andytaylor commented on PR #99: URL: https://github.com/apache/activemq-artemis-console/pull/99#issuecomment-3284362930 This looks good I will merge. just fyi if a user does get into trouble wrt too many items to load, the local storage can be reset in preferences -- This is an automated

Re: [PR] AMQ-9757: Upgrade to Apache POM 35 [activemq]

2025-09-11 Thread via GitHub
jbonofre merged PR #1482: URL: https://github.com/apache/activemq/pull/1482 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@activemq.

[PR] AMQ-9767: Upgrade to jmdns 3.6.2 [activemq]

2025-09-11 Thread via GitHub
jbonofre opened a new pull request, #1486: URL: https://github.com/apache/activemq/pull/1486 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-11 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2334176072 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyAcceptor.java: ## @@ -145,6 +146,8 @@ public class NettyAcceptor ext

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-11 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330602103 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -184,7 +208,9 @@ public void channelRead(ChannelHandlerC

Re: [PR] ARTEMIS-X Bump org.apache.groovy:groovy-all from 5.0.0 to 5.0.1 [activemq-artemis]

2025-09-11 Thread via GitHub
dependabot[bot] closed pull request #5917: ARTEMIS-X Bump org.apache.groovy:groovy-all from 5.0.0 to 5.0.1 URL: https://github.com/apache/activemq-artemis/pull/5917 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] ARTEMIS-4924 handle invalid messages in SNF queue [activemq-artemis]

2025-09-11 Thread via GitHub
jbertram commented on PR #5921: URL: https://github.com/apache/activemq-artemis/pull/5921#issuecomment-3282280383 In my opinion the question of which DLA to use isn't significant. The main thing is that we allow the bridge to function properly (i.e. by removing the offending message) and p

Re: [PR] ARTEMIS-4924 Proper handling of invalid messages in SNF queues [activemq-artemis]

2025-09-11 Thread via GitHub
jbertram closed pull request #5091: ARTEMIS-4924 Proper handling of invalid messages in SNF queues URL: https://github.com/apache/activemq-artemis/pull/5091 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] ARTEMIS-4924 Proper handling of invalid messages in SNF queues [activemq-artemis]

2025-09-11 Thread via GitHub
jbertram commented on PR #5091: URL: https://github.com/apache/activemq-artemis/pull/5091#issuecomment-3282183939 Replaced by #5921. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] ARTEMIS-4924 handle invalid messages in SNF queue [activemq-artemis]

2025-09-11 Thread via GitHub
jbertram commented on PR #5921: URL: https://github.com/apache/activemq-artemis/pull/5921#issuecomment-3282179479 This is replacing #5921. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spe

Re: [PR] ARTEMIS-5656 Prevent federation from creating self sustaining loops [activemq-artemis]

2025-09-11 Thread via GitHub
tabish121 commented on PR #5920: URL: https://github.com/apache/activemq-artemis/pull/5920#issuecomment-3282062887 All tests passing on CI -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spe

[PR] ARTEMIS-5656 Prevent federation from creating self sustaining loops [activemq-artemis]

2025-09-11 Thread via GitHub
tabish121 opened a new pull request, #5920: URL: https://github.com/apache/activemq-artemis/pull/5920 Prevent a federation source from considering bindings from its target to be demand that creates federation receivers in the opposite direction. This will prevent self sustaining local and r

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-11 Thread via GitHub
tabish121 commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2341680099 ## tests/e2e-tests/src/test/java/org/apache/activemq/artemis/tests/e2e/proxy_protocol/HAProxyTest.java: ## @@ -0,0 +1,334 @@ +/* + * Licensed to the Apache S

Re: [PR] WIP: [AMQ-9359] Jetty-12 conversion [activemq]

2025-09-11 Thread via GitHub
mattrpav commented on PR #1344: URL: https://github.com/apache/activemq/pull/1344#issuecomment-3281528725 UPDATE: TldCache error resolved by adding two jetty-ee9 dependencies to the distribution: jetty-ee9-annotations and jetty-ee9-plus. New error is missing tag libary ``` o

Re: [PR] WIP: [AMQ-9359] Jetty-12 conversion [activemq]

2025-09-11 Thread via GitHub
mattrpav commented on PR #1344: URL: https://github.com/apache/activemq/pull/1344#issuecomment-3281183012 @grgrzybek FYI: the org.eclipse.jetty/apache-jsp dependency (where error is being thrown) is transitive from org.eclipse.jetty.ee9/jetty-ee9-apache-jsp -- This is an automated message

Re: [PR] WIP: [AMQ-9359] Jetty-12 conversion [activemq]

2025-09-11 Thread via GitHub
mattrpav commented on PR #1344: URL: https://github.com/apache/activemq/pull/1344#issuecomment-3280871245 Current /admin console error ``` WARN | /admin/ org.apache.jasper.JasperException: /index.jsp (line: [1], column: [1]) Cannot invoke "org.apache.jasper.compiler.TldCache.ge

Re: [PR] ARTEMIS-X Bump org.apache.groovy:groovy-all from 5.0.0 to 5.0.1 [activemq-artemis]

2025-09-11 Thread via GitHub
dependabot[bot] commented on PR #5917: URL: https://github.com/apache/activemq-artemis/pull/5917#issuecomment-3280570436 Looks like org.apache.groovy:groovy-all is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] WIP: [AMQ-9359] Jetty-12 conversion [activemq]

2025-09-11 Thread via GitHub
mattrpav commented on PR #1344: URL: https://github.com/apache/activemq/pull/1344#issuecomment-3275569166 @coheigea I updated the list at the top of this PR. 1. @grgrzybek helped fix web socket STOMP tests, that pattern needs to be applied to the MQTT web socket tests. That will comp

Re: [PR] ARTEMIS-5654 - fix test broken by console upgrade [activemq-artemis]

2025-09-11 Thread via GitHub
gemmellr merged PR #5919: URL: https://github.com/apache/activemq-artemis/pull/5919 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@a

[PR] ARTEMIS-5654 - fix test broken by console upgrade [activemq-artemis]

2025-09-11 Thread via GitHub
andytaylor opened a new pull request, #5919: URL: https://github.com/apache/activemq-artemis/pull/5919 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsub

[PR] ARTEMIS-X Bump jline.version from 3.30.5 to 3.30.6 [activemq-artemis]

2025-09-10 Thread via GitHub
dependabot[bot] opened a new pull request, #5918: URL: https://github.com/apache/activemq-artemis/pull/5918 Bumps `jline.version` from 3.30.5 to 3.30.6. Updates `org.jline:jline-console` from 3.30.5 to 3.30.6 Release notes Sourced from https://github.com/jline/jline3/releases";>o

[PR] ARTEMIS-X Bump org.apache.groovy:groovy-all from 5.0.0 to 5.0.1 [activemq-artemis]

2025-09-10 Thread via GitHub
dependabot[bot] opened a new pull request, #5917: URL: https://github.com/apache/activemq-artemis/pull/5917 Bumps [org.apache.groovy:groovy-all](https://github.com/apache/groovy) from 5.0.0 to 5.0.1. Commits See full diff in https://github.com/apache/groovy/commits";>compare vi

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-10 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2337762318 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/HAProxyMessageHandler.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed to the A

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-10 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2337749293 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/HAProxyMessageHandler.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed to the A

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-10 Thread via GitHub
tabish121 commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2337553736 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/HAProxyMessageHandler.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed to the

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-10 Thread via GitHub
jbertram commented on PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#issuecomment-3275537103 I added negative tests as well in `org.apache.activemq.artemis.tests.e2e.proxy_protocol.HAProxyTest`: - `testNonProxiedConnectionToProxyAcceptor` - `testProxiedConnect

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-10 Thread via GitHub
gemmellr commented on PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#issuecomment-3275503139 I havent had a chance to look properly at this after all the changes yet...but skimming the tests, I only see spotted positive tests where it was expected to work and did. We s

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-10 Thread via GitHub
gemmellr commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2337148033 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyServerConnection.java: ## @@ -51,4 +58,55 @@ public void setSNIHostn

[PR] ARTEMIS-5651 Add missing synchronized tag to some brifge methods [activemq-artemis]

2025-09-10 Thread via GitHub
tabish121 opened a new pull request, #5913: URL: https://github.com/apache/activemq-artemis/pull/5913 Add synchronized tags to methods in the AMQP bridge from and to queue policy managers -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] ARTEMIS-5651 Add missing synchronized tag to some brifge methods [activemq-artemis]

2025-09-09 Thread via GitHub
tabish121 merged PR #5913: URL: https://github.com/apache/activemq-artemis/pull/5913 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@

[PR] ARTEMIS-5652 Bump jetty.version from 12.1.0 to 12.1.1 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram opened a new pull request, #5914: URL: https://github.com/apache/activemq-artemis/pull/5914 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubsc

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330718431 ## docs/user-manual/versions.adoc: ## @@ -20,6 +20,10 @@ https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&versio === Highlights

Re: [PR] ARTEMIS-5654 Bump artemis-console-version from 1.2.1 to 1.3.0 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram merged PR #5916: URL: https://github.com/apache/activemq-artemis/pull/5916 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@a

Re: [PR] ARTEMIS-X Bump artemis-console-version from 1.2.1 to 1.3.0 [activemq-artemis]

2025-09-09 Thread via GitHub
dependabot[bot] commented on PR #5905: URL: https://github.com/apache/activemq-artemis/pull/5905#issuecomment-3272185679 OK, I won't notify you again about this release, but will get in touch when a new version is available. You can also ignore all major, minor, or patch releases for a dep

Re: [PR] ARTEMIS-X Bump artemis-console-version from 1.2.1 to 1.3.0 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram closed pull request #5905: ARTEMIS-X Bump artemis-console-version from 1.2.1 to 1.3.0 URL: https://github.com/apache/activemq-artemis/pull/5905 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[PR] ARTEMIS-5654 Bump artemis-console-version from 1.2.1 to 1.3.0 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram opened a new pull request, #5916: URL: https://github.com/apache/activemq-artemis/pull/5916 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubsc

[PR] ARTEMIS-5653 Bump io.micrometer:micrometer-core from 1.15.3 to 1.15.4 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram opened a new pull request, #5915: URL: https://github.com/apache/activemq-artemis/pull/5915 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubsc

Re: [PR] ARTEMIS-X Bump io.micrometer:micrometer-core from 1.15.3 to 1.15.4 [activemq-artemis]

2025-09-09 Thread via GitHub
dependabot[bot] commented on PR #5910: URL: https://github.com/apache/activemq-artemis/pull/5910#issuecomment-3272152277 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor ve

Re: [PR] ARTEMIS-5653 Bump io.micrometer:micrometer-core from 1.15.3 to 1.15.4 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram merged PR #5915: URL: https://github.com/apache/activemq-artemis/pull/5915 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@a

Re: [PR] ARTEMIS-X Bump io.micrometer:micrometer-core from 1.15.3 to 1.15.4 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram closed pull request #5910: ARTEMIS-X Bump io.micrometer:micrometer-core from 1.15.3 to 1.15.4 URL: https://github.com/apache/activemq-artemis/pull/5910 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] ARTEMIS-X Bump jetty.version from 12.1.0 to 12.1.1 [activemq-artemis]

2025-09-09 Thread via GitHub
dependabot[bot] commented on PR #5912: URL: https://github.com/apache/activemq-artemis/pull/5912#issuecomment-3272146671 OK, I won't notify you again about this release, but will get in touch when a new version is available. You can also ignore all major, minor, or patch releases for a dep

Re: [PR] ARTEMIS-X Bump jetty.version from 12.1.0 to 12.1.1 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram closed pull request #5912: ARTEMIS-X Bump jetty.version from 12.1.0 to 12.1.1 URL: https://github.com/apache/activemq-artemis/pull/5912 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the s

Re: [PR] ARTEMIS-5652 Bump jetty.version from 12.1.0 to 12.1.1 [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram merged PR #5914: URL: https://github.com/apache/activemq-artemis/pull/5914 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@a

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2334175593 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -184,7 +208,9 @@ public void channelRead(ChannelHandlerC

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram commented on PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#issuecomment-3271856362 FYI - test-suite is good on this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go t

Re: [PR] ARTEMIS-5650 AMQP Federation receivers prefer modified dispositions b… [activemq-artemis]

2025-09-09 Thread via GitHub
gemmellr merged PR #5909: URL: https://github.com/apache/activemq-artemis/pull/5909 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@a

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-09 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2334172618 ## tests/e2e-tests/src/test/java/org/apache/activemq/artemis/tests/e2e/proxy/HAProxyTest.java: ## @@ -0,0 +1,253 @@ +/* + * Licensed to the Apache Software Fo

Re: [PR] ARTEMIS-X Bump netty.version from 4.1.126.Final to 4.1.127.Final [activemq-artemis]

2025-09-09 Thread via GitHub
gemmellr commented on PR #5911: URL: https://github.com/apache/activemq-artemis/pull/5911#issuecomment-3270219414 @dependabot rebase -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[PR] ARTEMIS-X Bump netty.version from 4.1.126.Final to 4.1.127.Final [activemq-artemis]

2025-09-08 Thread via GitHub
dependabot[bot] opened a new pull request, #5911: URL: https://github.com/apache/activemq-artemis/pull/5911 Bumps `netty.version` from 4.1.126.Final to 4.1.127.Final. Updates `io.netty:netty-buffer` from 4.1.126.Final to 4.1.127.Final Commits https://github.com/netty/netty/co

[PR] ARTEMIS-X Bump jetty.version from 12.1.0 to 12.1.1 [activemq-artemis]

2025-09-08 Thread via GitHub
dependabot[bot] opened a new pull request, #5912: URL: https://github.com/apache/activemq-artemis/pull/5912 Bumps `jetty.version` from 12.1.0 to 12.1.1. Updates `org.eclipse.jetty:jetty-bom` from 12.1.0 to 12.1.1 Release notes Sourced from https://github.com/jetty/jetty.project/r

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330724672 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyServerConnection.java: ## @@ -51,4 +58,55 @@ public void setSNIHostn

[PR] ARTEMIS-5650 AMQP Federation receivers prefer modified dispositions b… [activemq-artemis]

2025-09-08 Thread via GitHub
tabish121 opened a new pull request, #5909: URL: https://github.com/apache/activemq-artemis/pull/5909 …y default AMQP Federation receiver links configure themselves to a default of sending a modified disposition for address full errors to allow the remote sender to redeliver the mess

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
gemmellr commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330860243 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -184,7 +208,9 @@ public void channelRead(ChannelHandlerC

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
gemmellr commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330856027 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -147,11 +156,26 @@ public void channelInactive(ChannelHa

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330646337 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -147,11 +156,26 @@ public void channelInactive(ChannelHa

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330646337 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -147,11 +156,26 @@ public void channelInactive(ChannelHa

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330494969 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/view/ConsumerField.java: ## @@ -33,6 +33,8 @@ public enum ConsumerField {

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
jbertram commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330386334 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java: ## @@ -147,11 +156,26 @@ public void channelInactive(ChannelHa

Re: [PR] ARTEMIS-3915 Support PROXY protocol [activemq-artemis]

2025-09-08 Thread via GitHub
gemmellr commented on code in PR #5908: URL: https://github.com/apache/activemq-artemis/pull/5908#discussion_r2330145498 ## artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyServerConnection.java: ## @@ -51,4 +58,55 @@ public void setSNIHostn

Re: [PR] AMQ-9740: Fixes "org.apache.commons.lang3.StringEscapeUtils cannot be resolved to a type" in the Web Console. [activemq]

2025-09-08 Thread via GitHub
mattrpav commented on PR #1469: URL: https://github.com/apache/activemq/pull/1469#issuecomment-3266206465 @coheigea the issue only impacted 'main' branch. At the time of JIRA reporting, the main may have been versioned 6.1.8-SNAPSHOT. I cleaned-up the _affected_ and _fixed_ fields in the JI

  1   2   3   4   5   6   7   8   9   10   >