<div dir="ltr">Sorry<br><br>Before:<br><br><font face="monospace">     * {@link Filter} modules may store arbitrary objects with {@code HttpExchange}</font><br><font face="monospace">     * instances as an out-of-band communication mechanism. Other filters</font><br><font face="monospace">     * or the exchange handler may then access these objects.</font><br><br><font face="arial, sans-serif">Bungled the copy-paste</font></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Dec 5, 2024 at 6:49 AM Ethan McCue <<a href="mailto:ethan@mccue.dev">ethan@mccue.dev</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi all,<br><br>This change (<a href="https://github.com/openjdk/jdk/commit/40ae4699622cca72830acd146b7b5c4efd5a43ec" target="_blank">https://github.com/openjdk/jdk/commit/40ae4699622cca72830acd146b7b5c4efd5a43ec</a>) makes the Jetty implementation of the SPI be no longer fit the Javadoc.<br><br>HttpContexts are not per-request while the previous Javadoc implied that the attribute mechanism on exchanges was.<br><br>Before:<br><br><font face="monospace">     * Sets an attribute with the given {@code name} and {@code value} in this exchange's<br>     * {@linkplain HttpContext#getAttributes() context attributes}.<br>     * or the exchange handler may then access these objects.</font><br><br>After:<br><br><font face="monospace">     * Sets an attribute with the given {@code name} and {@code value} in this exchange's<br>     * {@linkplain HttpContext#getAttributes() context attributes}.<br>     *<br>     * @apiNote {@link Filter} modules may store arbitrary objects as attributes through<br>     * {@code HttpExchange} instances as an out-of-band communication mechanism. Other filters<br>     * or the exchange handler may then access these objects.<br></font><br>The Jetty implementation, I think rightfully, assumed that this context was per-request and implemented it as so.<br><br><a href="https://github.com/jetty/jetty.project/blob/jetty-12.0.x/jetty-core/jetty-http-spi/src/main/java/org/eclipse/jetty/http/spi/JettyHttpExchangeDelegate.java#L223" target="_blank">https://github.com/jetty/jetty.project/blob/jetty-12.0.x/jetty-core/jetty-http-spi/src/main/java/org/eclipse/jetty/http/spi/JettyHttpExchangeDelegate.java#L223</a><br><br>As such, I don't think simply a javadoc change as a resolution to these issues is applicable<br><br><a href="https://bugs.openjdk.org/browse/JDK-8345233" target="_blank">https://bugs.openjdk.org/browse/JDK-8345233</a><br><a href="https://bugs.openjdk.org/browse/JDK-8235786" target="_blank">https://bugs.openjdk.org/browse/JDK-8235786</a><br><br></div>
</blockquote></div>