fix: rework null checks with nullaway [skip ci]

Author: mmoayyed

.github/workflows/release.yml

@@ -67,7 +67,7 @@ jobs:
         run: find ./ci -type f -name "*.sh" -exec chmod +x "{}" \; && ./ci/init-build.sh
       - name: Configure PGP
         run: echo "$PGP_PRIVATE_KEY" | gpg --batch --import
-      - name: Release CAS
+      - name: Release CAS ${{ inputs.releaseVersion }}
         run: |
           RELEASE_VERSION="${{ inputs.releaseVersion }}"
           

core/cas-server-core/src/main/java/org/apereo/cas/DefaultCentralAuthenticationService.java

@@ -68,14 +68,10 @@
  */
 @Slf4j
 public class DefaultCentralAuthenticationService extends AbstractCentralAuthenticationService {
-    @Serial
-    private static final long serialVersionUID = -8943828074939533986L;
-
     public DefaultCentralAuthenticationService(final CentralAuthenticationServiceContext context) {
         super(context);
     }
 
-
     @Audit(
         action = AuditableActions.TICKET_GRANTING_TICKET,
         actionResolverName = AuditActionResolvers.CREATE_TICKET_GRANTING_TICKET_RESOLVER,
@@ -257,7 +253,7 @@ public Assertion validateServiceTicket(final String serviceTicketId, final Servi
             val ticketGrantingTicket = (TicketGrantingTicket) serviceTicket.getTicketGrantingTicket();
             var authentication = serviceTicket.isStateless()
                 ? serviceTicket.getAuthentication()
-                : ticketGrantingTicket.getRoot().getAuthentication();
+                : Objects.requireNonNull(ticketGrantingTicket).getRoot().getAuthentication();
 
             authentication = getAuthenticationSatisfiedByPolicy(authentication, selectedService, Objects.requireNonNull(registeredService));
             Objects.requireNonNull(authentication, "Authentication cannot be determined for service ticket validation");
@@ -309,7 +305,7 @@ public Assertion validateServiceTicket(final String serviceTicketId, final Servi
 
             val assertionContext = serviceTicket.isStateless()
                 ? CollectionUtils.<String, Serializable>wrap(Principal.class.getName(), authentication.getPrincipal().getId())
-                : CollectionUtils.<String, Serializable>wrap(TicketGrantingTicket.class.getName(), ticketGrantingTicket.getRoot().getId());
+                : CollectionUtils.<String, Serializable>wrap(TicketGrantingTicket.class.getName(), Objects.requireNonNull(ticketGrantingTicket).getRoot().getId());
 
             val assertion = DefaultAssertionBuilder.builder()
                 .primaryAuthentication(finalAuthentication)
@@ -318,7 +314,7 @@ public Assertion validateServiceTicket(final String serviceTicketId, final Servi
                 .registeredService(registeredService)
                 .authentications(serviceTicket.isStateless()
                     ? List.of(Objects.requireNonNull(serviceTicket.getAuthentication()))
-                    : ticketGrantingTicket.getChainedAuthentications())
+                    : Objects.requireNonNull(ticketGrantingTicket).getChainedAuthentications())
                 .newLogin(((RenewableServiceTicket) serviceTicket).isFromNewLogin())
                 .stateless(serviceTicket.isStateless())
                 .context(assertionContext)
@@ -376,7 +372,8 @@ public Ticket createProxyGrantingTicket(final String serviceTicketId,
                     val addedTicket = configurationContext.getTicketRegistry().addTicket(proxyGrantingTicket);
                     LOGGER.debug("Generated proxy granting ticket [{}] based off of [{}]", proxyGrantingTicket, serviceTicketId);
                     if (!serviceTicket.isStateless()) {
-                        configurationContext.getTicketRegistry().updateTicket(serviceTicket.getTicketGrantingTicket());
+                        configurationContext.getTicketRegistry()
+                            .updateTicket(Objects.requireNonNull(serviceTicket.getTicketGrantingTicket()));
                     }
                     val clientInfo = ClientInfoHolder.getClientInfo();
                     doPublishEvent(new CasProxyGrantingTicketCreatedEvent(this, addedTicket, clientInfo));

support/cas-server-support-openrewrite/recipes/META-INF/rewrite/cas800.yml

@@ -29,7 +29,7 @@ recipeList:
       overwrite: true
       filePattern: 'gradle.properties'
   - org.openrewrite.gradle.UpdateGradleWrapper:
-      version: 9.3.0-rc-3
+      version: 9.3.0
       addIfMissing: true
   - org.openrewrite.gradle.AddProperty:
       key: tomcatVersion