[JENKINS-63355] Prevent NPEs when library name is not provided

src/main/java/org/jenkinsci/plugins/workflow/libs/LibraryConfiguration.java

@@ -63,13 +63,17 @@ public class LibraryConfiguration extends AbstractDescribableImpl<LibraryConfigu
 
     @DataBoundConstructor public LibraryConfiguration(String name, LibraryRetriever retriever) {
         this.name = Util.fixEmptyAndTrim(name);
+        if(this.name == null) {
+            throw new IllegalArgumentException("You must enter a library name");
+        }
         this.retriever = retriever;
     }
 
     /**
      * Library name.
      * Should match {@link Library#value}, up to the first occurrence of {@code @}, if any.
      */
+    @NonNull
     public String getName() {
         return name;
     }
@@ -173,6 +177,9 @@ public FormValidation doCheckName(@QueryParameter String name) {
 
         @RequirePOST
         public FormValidation doCheckDefaultVersion(@AncestorInPath Item context, @QueryParameter String defaultVersion, @QueryParameter boolean implicit, @QueryParameter boolean allowVersionOverride, @QueryParameter String name) {
+            if (name.isEmpty()) {
+                return FormValidation.error("You must enter a name.");
+            }
             if (defaultVersion.isEmpty()) {
                 if (implicit) {
                     return FormValidation.error("If you load a library implicitly, you must specify a default version.");
@@ -184,7 +191,7 @@ public FormValidation doCheckDefaultVersion(@AncestorInPath Item context, @Query
             } else {
                 for (LibraryResolver resolver : ExtensionList.lookup(LibraryResolver.class)) {
                     for (LibraryConfiguration config : resolver.fromConfiguration(Stapler.getCurrentRequest())) {
-                        if (config.getName().equals(name)) {
+                        if (name.equals(config.getName())) {
                             return config.getRetriever().validateVersion(name, defaultVersion, context);
                         }
                     }

src/main/java/org/jenkinsci/plugins/workflow/libs/LibraryStep.java

@@ -168,6 +168,9 @@ public static class Execution extends AbstractSynchronousNonBlockingStepExecutio
         @Override protected LoadedClasses run() throws Exception {
             String[] parsed = LibraryAdder.parse(step.identifier);
             String name = parsed[0], version = parsed[1];
+            if (name == null) {
+                throw new AbortException("No library name provided");
+            }
             boolean trusted = false;
             Boolean changelog = step.getChangelog();
             LibraryCachingConfiguration cachingConfiguration = null;
@@ -177,7 +180,7 @@ public static class Execution extends AbstractSynchronousNonBlockingStepExecutio
             if (retriever == null) {
                 for (LibraryResolver resolver : ExtensionList.lookup(LibraryResolver.class)) {
                     for (LibraryConfiguration cfg : resolver.forJob(run.getParent(), Collections.singletonMap(name, version))) {
-                        if (cfg.getName().equals(name)) {
+                        if (name.equals(cfg.getName())) {
                             retriever = cfg.getRetriever();
                             trusted = resolver.isTrusted();
                             version = cfg.defaultedVersion(version);

src/main/resources/org/jenkinsci/plugins/workflow/libs/LibraryStep/config.jelly

@@ -26,7 +26,7 @@ THE SOFTWARE.
 <?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">
     <f:entry field="identifier" title="${%Name and optional version}">
-        <f:textbox/>
+        <f:textbox clazz="required"/>
     </f:entry>
     <f:entry field="changelog" title="${%Include recent changes in jobs}">
         <f:checkbox default="true"/>

src/test/java/org/jenkinsci/plugins/workflow/libs/LibraryConfigurationTest.java

@@ -24,7 +24,7 @@
 
 package org.jenkinsci.plugins.workflow.libs;
 
-import java.util.Collections;
+import java.util.List;
 
 import hudson.plugins.git.GitSCM;
 import org.hamcrest.Matchers;
@@ -69,29 +69,24 @@ public class LibraryConfigurationTest {
     }
 
     @Issue("JENKINS-59527")
-    @Test public void emptyStringDefaultVersionAndName() {
-        String libraryName = "";
-        String defaultVersion = "";
-
-        LibraryConfiguration cfg = new LibraryConfiguration(libraryName, new SCMRetriever(new GitSCM("https://phony.jenkins.io/bar.git")));
-        cfg.setDefaultVersion(defaultVersion);
-
-        assertNull(cfg.getName());
+    @Test public void emptyDefaultVersion() {
+        LibraryConfiguration cfg = new LibraryConfiguration("test", new SCMRetriever(new GitSCM("https://phony.jenkins.io/bar.git")));
+        cfg.setDefaultVersion("");
         assertNull(cfg.getDefaultVersion());
-    }
-
-    @Issue("JENKINS-59527")
-    @Test public void nullDefaultVersionAndName() {
-        String libraryName = null;
-        String defaultVersion = null;
-
-        LibraryConfiguration cfg = new LibraryConfiguration(libraryName, new SCMRetriever(new GitSCM("https://phony.jenkins.io/bar.git")));
-        cfg.setDefaultVersion(defaultVersion);
 
-        assertNull(cfg.getName());
+        cfg = new LibraryConfiguration("test", new SCMRetriever(new GitSCM("https://phony.jenkins.io/bar.git")));
+        cfg.setDefaultVersion(null);
         assertNull(cfg.getDefaultVersion());
     }
 
-
+    @Issue("JENKINS-63355")
+    @Test public void emptyNameCannotBeSaved() throws Exception {
+        assertThrows(IllegalArgumentException.class, () -> GlobalLibraries.get().setLibraries(List.of(
+            new LibraryConfiguration(null, new SCMRetriever(new GitSCM("https://phony.jenkins.io/bar.git")))
+        )));
+        assertThrows(IllegalArgumentException.class, () -> GlobalLibraries.get().setLibraries(List.of(
+            new LibraryConfiguration("", new SCMRetriever(new GitSCM("https://phony.jenkins.io/bar.git")))
+        )));
+    }
 
 }