diff --git a/spring-integration-core/src/main/java/org/springframework/integration/util/UUIDConverter.java b/spring-integration-core/src/main/java/org/springframework/integration/util/UUIDConverter.java index 2c4283ecc4..14ce2872fa 100644 --- a/spring-integration-core/src/main/java/org/springframework/integration/util/UUIDConverter.java +++ b/spring-integration-core/src/main/java/org/springframework/integration/util/UUIDConverter.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2019 the original author or authors. + * Copyright 2002-2023 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -19,8 +19,9 @@ package org.springframework.integration.util; import java.io.ByteArrayOutputStream; import java.io.IOException; import java.io.ObjectOutputStream; -import java.io.UnsupportedEncodingException; +import java.nio.charset.StandardCharsets; import java.util.UUID; +import java.util.regex.Pattern; import org.springframework.core.convert.converter.Converter; import org.springframework.util.ClassUtils; @@ -30,16 +31,22 @@ import org.springframework.util.ClassUtils; * * @author Dave Syer * @author Gary Russell + * @author Christian Tzolov */ public class UUIDConverter implements Converter { + /** + * @deprecated since 6.0.8 as it is not used internally by the UUIDConverter. The internal implementation relies, now, + * on StandardCharsets.UTF_8 instead. + */ + @Deprecated public static final String DEFAULT_CHARSET = "UTF-8"; + private static final Pattern UUID_REGEX = Pattern + .compile("^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$"); /** - * Convert the input to a UUID using the convenience method - * {@link #getUUID(Object)}. - * + * Convert the input to a UUID using the convenience method {@link #getUUID(Object)}. * @see org.springframework.core.convert.converter.Converter#convert(java.lang.Object) */ @Override @@ -52,17 +59,12 @@ public class UUIDConverter implements Converter { * * If none of the above applies there will be an exception trying to serialize. - * * @param input an Object * @return a UUID constructed from the input */ @@ -74,28 +76,16 @@ public class UUIDConverter implements Converter { return (UUID) input; } if (input instanceof String) { - try { - return UUID.fromString((String) input); + String inputText = (String) input; + if (isValidUuidStringRepresentation(inputText)) { + return UUID.fromString(inputText); } - catch (Exception e) { - try { - return UUID.nameUUIDFromBytes(((String) input).getBytes(DEFAULT_CHARSET)); - } - catch (UnsupportedEncodingException ex) { - IllegalStateException exception = - new IllegalStateException("Cannot convert String using charset=" + DEFAULT_CHARSET, ex); - exception.addSuppressed(e); - throw exception; // NOSONAR - added to suppressed exceptions - } + else { + return UUID.nameUUIDFromBytes((inputText).getBytes(StandardCharsets.UTF_8)); } } if (ClassUtils.isPrimitiveOrWrapper(input.getClass())) { - try { - return UUID.nameUUIDFromBytes(input.toString().getBytes(DEFAULT_CHARSET)); - } - catch (UnsupportedEncodingException e) { - throw new IllegalStateException("Cannot convert primitive using charset=" + DEFAULT_CHARSET, e); - } + return UUID.nameUUIDFromBytes(input.toString().getBytes(StandardCharsets.UTF_8)); } byte[] bytes = serialize(input); return UUID.nameUUIDFromBytes(bytes); @@ -115,4 +105,8 @@ public class UUIDConverter implements Converter { return stream.toByteArray(); } + private static boolean isValidUuidStringRepresentation(String uuid) { + return UUID_REGEX.matcher(uuid).matches(); + } + }