[resteasy-commits] [resteasy/resteasy] a77b9f: Use getHostName over getHostString when setting RE...

Branch: refs/heads/5.0 Home: https://github.com/resteasy/resteasy Commit: a77b9f2dacb1d30dae3a50d6a5a6eb7b1d06069d https://github.com/resteasy/resteasy/commit/a77b9f2dacb1d30dae3a50d6a5a6e... Author: Samuel Cox <sacox(a)paypal.com&gt; Date: 2022-10-07 (Fri, 07 Oct 2022) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java Log Message: ----------- Use getHostName over getHostString when setting RE's URI object. We want to make a greater effort at capturing the actual hostname used by the client. Commit: 3d5c9f2d487a238f519629fc574a00208294a9a9 https://github.com/resteasy/resteasy/commit/3d5c9f2d487a238f519629fc574a0... Author: Samuel Cox <crankydillo(a)gmail.com&gt; Date: 2022-10-11 (Tue, 11 Oct 2022) Changed paths: M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java Log Message: ----------- Remove a stale comment. Commit: 48db1a291a719f2554d965663734b0d02be471de https://github.com/resteasy/resteasy/commit/48db1a291a719f2554d965663734b... Author: Samuel Cox <crankydillo(a)gmail.com&gt; Date: 2022-10-11 (Tue, 11 Oct 2022) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java Log Message: ----------- The request URI should be governed by the client, which means we need to read it from the Host header. Not sure why we didn't just copy what the other adapters did. Honestly, some of that should probably be abstracted _if_ reading from Host header is truly the right thing to do. I _think_ we shied away from this because it felt wrong to return 'unknown'. While that still feels wrong, I'm not sure what is a better option, so copying. I briefly thought about returning the IP address, which is what was happening before this effort to _not_ return the IP address! Commit: f61707e55577a1695b9908ecdae1902f7f54411f https://github.com/resteasy/resteasy/commit/f61707e55577a1695b9908ecdae19... Author: Samuel Cox <crankydillo(a)gmail.com&gt; Date: 2022-12-01 (Thu, 01 Dec 2022) Changed paths: M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java R server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java Log Message: ----------- Use the IP address if the Host header is invalid. Commit: 31913ed34c99d7d339f74d8c7dc426d6841d2222 https://github.com/resteasy/resteasy/commit/31913ed34c99d7d339f74d8c7dc42... Author: Samuel Cox <crankydillo(a)gmail.com&gt; Date: 2022-12-03 (Sat, 03 Dec 2022) Changed paths: M server-adapters/resteasy-reactor-netty/pom.xml M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java Log Message: ----------- Throw IllegalArgumentException if the host address cannot be extracted.. from the request. We discussed 'asserting' here. Maybe that is the best way. I just haven't seen anyone do that in quite a while. Commit: 4ee135b2d665e2154b86771130ed159f58b65fce https://github.com/resteasy/resteasy/commit/4ee135b2d665e2154b86771130ed1... Author: Samuel Cox <crankydillo(a)gmail.com&gt; Date: 2022-12-09 (Fri, 09 Dec 2022) Changed paths: M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java Log Message: ----------- Remove an empty comment. Commit: 417393f78f1b4b28da0de104aea38df60498da94 https://github.com/resteasy/resteasy/commit/417393f78f1b4b28da0de104aea38... Author: James R. Perkins <jperkins(a)redhat.com&gt; Date: 2022-12-12 (Mon, 12 Dec 2022) Changed paths: M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java Log Message: ----------- [RESTEASY-3261] Fix the UriInfoTest. https://issues.redhat.com/browse/RESTEASY-3261 Signed-off-by: James R. Perkins <jperkins(a)redhat.com&gt; Commit: f31351a63187989ad14adc62f8980988084d7f0b https://github.com/resteasy/resteasy/commit/f31351a63187989ad14adc62f8980... Author: James R. Perkins <jperkins(a)redhat.com&gt; Date: 2022-12-12 (Mon, 12 Dec 2022) Changed paths: M server-adapters/resteasy-reactor-netty/pom.xml M server-adapters/resteasy-reactor-netty/src/main/java/org/jboss/resteasy/plugins/server/reactor/netty/ReactorNettyJaxrsServer.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/plugins/server/reactor/netty/UriInfoTest.java R server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/HeaderEmptyHostTest.java M server-adapters/resteasy-reactor-netty/src/test/java/org/jboss/resteasy/test/NettyTest.java Log Message: ----------- Merge pull request #3361 from jamezp/reactor-netty-server-uriinfo Use the request's Host header when setting RE's URI object. Compare: https://github.com/resteasy/resteasy/compare/6149edbe0ce2...f31351a63187