Joakim Erdfelt

Results 489 comments of Joakim Erdfelt

> 2\. Add more information to the exception message, such as `Invalid SNI. localhost is not allowed` or `Invalid SNI IP address literals are not allowed`, etc. This is the...

We cannot fix this. `java.util.ServiceLoader.Provider` was added in OpenJDK 9. https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/util/ServiceLoader.Provider.html There many components that rely on this, and Jetty is only a small part of this reliance. This is...

Closing, this works for others on Android. You have to configure your project to compile on Java 11. This error only shows up if you haven't done that properly.

The `ServletContextHandler` has it's own temp directory. That follows the servlet spec rules, and does not use any Server level behavior, if it's undeclared by the ServletContext, then it is...

The `jakarta.servlet.context.tempdir` attribute is configured by calling `ServletContextHandler.setTempDirectory(File)`. Which calls the `ServletContextHandler.setAttribute(ServletContext.TEMPDIR, new File("/absolute/path/to/the/context/specific/temp/directory"))` https://github.com/jetty/jetty.project/blob/a683690678fd0323918bb5079341d10ad8979c89/jetty-ee10/jetty-ee10-servlet/src/main/java/org/eclipse/jetty/ee10/servlet/ServletContextHandler.java#L294-L298 Without calling that method, you have no temp directory for that `ServletContext`, and no attribute...

A proposal PR #11397 from @lorban has been submitted to attempt to address the contract issue discovered with Server.getContext().getTempDirectory(). Fixing that contract issues causes a new issue. As _each_ `ServletContextHandler`...

@filiphr another point of difference between Jetty 11 and Jetty 12 is the changes in the servlet spec with regards to the `jakarta.servlet.MultipartConfigElement` This section of javadoc was added in...

> But that did not work as each context should have its own temp directory I still feel that every `ServletContextHandler` should have it's own private temp directory (as the...

> @lorban @joakime please see my comment on the original issue. > > The only change we need to make is to ensure that `ScopedContext` falls back to `java.io.tmpdir` after...