[java-idp-plugin-jetty] 02/03: Add annotations to some parameters

[Rod Widdowson]

This is an automated email from the git hooks/post-receive script.

rdw pushed a commit to branch main
in repository java-idp-plugin-jetty.

View the commit online:
https://git.shibboleth.net/view/?p=java-idp-plugin-jetty.git;a=commit;h=737528de3f77304d4c194160a7e08133ddb4b64f

commit 737528de3f77304d4c194160a7e08133ddb4b64f
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Wed Nov 12 17:17:43 2025 +0000

    Add annotations to some parameters
---
 .../main/java/net/shibboleth/idp/module/jetty/JettyModule.java | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/jetty-impl/src/main/java/net/shibboleth/idp/module/jetty/JettyModule.java b/jetty-impl/src/main/java/net/shibboleth/idp/module/jetty/JettyModule.java
index 9bc5827..1db06d1 100644
--- a/jetty-impl/src/main/java/net/shibboleth/idp/module/jetty/JettyModule.java
+++ b/jetty-impl/src/main/java/net/shibboleth/idp/module/jetty/JettyModule.java
@@ -68,7 +68,7 @@ public class JettyModule extends PluginIdPModule {
     /** {@inheritDoc} */
     @Override
     @Nonnull
-    public Map<ModuleResource, ResourceResult> enable(@Nonnull ModuleContext moduleContext) throws ModuleException {
+    public Map<ModuleResource, ResourceResult> enable(@Nonnull final ModuleContext moduleContext) throws ModuleException {
         final Map<ModuleResource, ResourceResult> result = super.enable(moduleContext);
         final Path idpHome = Path.of(moduleContext.getInstallLocation());
         final Path keystore= idpHome.resolve("credentials").resolve("idp-userfacing.p12");
@@ -87,7 +87,7 @@ public class JettyModule extends PluginIdPModule {
      * 
      * @throws ModuleException if we cannot find the logback version or get an IO issue
      */
-    private void handleLogbackJars(final @Nonnull Path idpHome, final @Nonnull @NotEmpty String jettyBaseName) throws ModuleException {
+    private void handleLogbackJars(@Nonnull final Path idpHome, @Nonnull @NotEmpty final String jettyBaseName) throws ModuleException {
 
         final Path jettyBaseDir = idpHome.resolve(jettyBaseName);
         final Path jarTargetDir = jettyBaseDir.resolve("lib").resolve("logging");
@@ -121,7 +121,7 @@ public class JettyModule extends PluginIdPModule {
      * 
      * @throws IOException if the copy fails
      */
-    private void copyLogbackJar(Path sourceDir, Path targetDir, String jarName) throws IOException {
+    private void copyLogbackJar(final @Nonnull Path sourceDir, @Nonnull final Path targetDir, @Nonnull final @NotEmpty String jarName) throws IOException {
 
         final File targetFile = targetDir.resolve(jarName).toFile();
 
@@ -153,7 +153,7 @@ public class JettyModule extends PluginIdPModule {
      * 
      * @throws ModuleException if the lookup failed
      */
-    private String inferLogbackVersion(Path distLibDir) throws ModuleException {
+    private String inferLogbackVersion(@Nonnull final Path distLibDir) throws ModuleException {
 
         try (DirectoryStream<Path> stream = Files.newDirectoryStream(distLibDir, "logback-classic-*.jar")) {
             Iterator<Path> i = stream.iterator();
@@ -203,7 +203,7 @@ public class JettyModule extends PluginIdPModule {
      * @param addr what to look at
      * @return true unless the name is the canonical name
      */
-    private boolean hasHostName(final InetAddress addr) {
+    private boolean hasHostName(@Nonnull final InetAddress addr) {
         return !addr.getHostAddress().equals(addr.getCanonicalHostName());
     }
 

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.