AB#81518 No longer log about null values.

src/main/java/uk/ac/ox/ctl/ltiauth/JWTService.java

@@ -33,7 +33,6 @@ public JWTService(JWTSigner signer, JWTStore store, LtiSettings ltiSettings) {
      */
     public String createJWT(OAuth2AuthenticationToken token, String toolSupportUrl) {
         JWTClaimsSet.Builder builder = new JWTClaimsSet.Builder();
-        checkForNullCustomValues(token);
         token.getPrincipal().getAttributes().forEach(builder::claim);
         builder
                 // Save the original issuer, this is needed for deep linking where we need to know who the audience is.
@@ -49,21 +48,6 @@ public String createJWT(OAuth2AuthenticationToken token, String toolSupportUrl)
             throw new BadCredentialsException("Failed to create new JWT");
         }
     }
-
-    void checkForNullCustomValues(OAuth2AuthenticationToken token) {
-        // Checks if any of the launches we handle currently get a null value
-        // This is only needed for a short time while we are checking if there's anything we should be looking
-        // at. Once Canvas has released the change to production we can remove this as we will be seeing the 
-        // errors.
-        Object claim = token.getPrincipal().getAttribute("https://purl.imsglobal.org/spec/lti/claim/custom");
-        if (claim instanceof Map custom) {
-            custom.forEach((key, value) -> {
-                if (value == null) {
-                    log.error("Custom attribute {} is null for client registration ID: {}", key, token.getAuthorizedClientRegistrationId());
-                }
-            });
-        }
-    }
 
     public String store(Object token) {
         return store.store(token);