There seems to be a section of wrong logic in org.springframework.boot.loader.jar.Handler#captureJarContextUrl

[aoyvx]

Looking at the following code, handlers will never be null.

spring-boot/spring-boot-project/spring-boot-tools/spring-boot-loader/src/main/java/org/springframework/boot/loader/jar/Handler.java

Lines 414 to 440 in 47516b5

	static void captureJarContextUrl() {
	if (canResetCachedUrlHandlers()) {
	String handlers = System.getProperty(PROTOCOL_HANDLER, "");
	try {
	System.clearProperty(PROTOCOL_HANDLER);
	try {
	resetCachedUrlHandlers();
	jarContextUrl = new URL("jar:file:context.jar!/");
	URLConnection connection = jarContextUrl.openConnection();
	if (connection instanceof JarURLConnection) {
	jarContextUrl = null;
	}
	}
	catch (Exception ex) {
	}
	}
	finally {
	if (handlers == null) {
	System.clearProperty(PROTOCOL_HANDLER);
	}
	else {
	System.setProperty(PROTOCOL_HANDLER, handlers);
	}
	}
	resetCachedUrlHandlers();
	}
	}

Therefore, whether the branch handlers==null can be deleted.

[aoyvx]

If this behavior does not meet expectations, please assign the issue to me.

[philwebb]

Good spot @aoyvx, I think we should be calling String handlers = System.getProperty(PROTOCOL_HANDLER); without the default value. I'll assign the issue to you. Thanks!

[philwebb]

Closing in favor of PR #31870. Thanks @aoyvx!