Apache DolphinScheduler文件上传扩展名校验问题分析与修复

问题背景

在Apache DolphinScheduler的工作流管理系统中，用户在上传资源文件时可能会遇到"file extension cannot not change"的错误提示。这个问题主要出现在重新上传文件时，系统对文件扩展名的校验逻辑存在缺陷。

问题现象

当用户尝试重新上传一个资源文件时，系统会抛出异常提示"文件扩展名不能更改"，即使实际上用户并没有修改文件扩展名。这个问题影响了正常的工作流资源管理功能。

问题根源分析

通过代码审查发现，问题的根源在于文件扩展名校验逻辑的实现方式不够健壮。原始代码在获取上传文件的文件名时，没有正确处理不同来源的文件名获取方式：

1. 对于新上传的文件，应该使用getOriginalFilename()方法获取原始文件名
2. 对于已存在的文件，可能需要使用getName()方法获取文件名
3. 代码没有对这两种情况进行区分处理，导致在某些情况下获取的文件名不正确

技术解决方案

为了解决这个问题，我们需要改进文件扩展名的校验逻辑。正确的实现应该：

1. 优先使用getOriginalFilename()获取上传文件的原始名称
2. 如果原始名称为空，则回退到使用getName()方法
3. 将获取到的文件名与目标路径的文件扩展名进行比对

改进后的代码逻辑如下：

if (!Objects.equals(
        Files.getFileExtension(
                file.getOriginalFilename() == null ? file.getName() : file.getOriginalFilename()),
        Files.getFileExtension(updateFileDto.getFileAbsolutePath()))) {
    throw new ServiceException("file extension cannot not change");
}

实现原理

这个修复方案的核心在于：

1. 多途径获取文件名：通过file.getOriginalFilename() == null ? file.getName() : file.getOriginalFilename()这一条件表达式，确保无论文件来源如何都能正确获取文件名
2. 扩展名比对：使用Files.getFileExtension()方法提取文件扩展名进行比对
3. 异常处理：当扩展名不匹配时抛出明确的业务异常

影响范围

该修复主要影响以下功能场景：

1. 资源文件的上传功能
2. 资源文件的重新上传功能
3. 工作流定义中引用的资源文件更新

最佳实践建议

对于使用Apache DolphinScheduler的开发者和用户，在处理文件上传时应注意：

1. 确保上传文件的扩展名与预期一致
2. 如果需要修改文件类型，应先删除旧文件再上传新文件
3. 对于自定义的文件处理逻辑，应参考这一修复方案实现健壮的扩展名校验

总结

这个问题的修复不仅解决了文件重新上传时的错误提示问题，更重要的是提供了一种健壮的文件名处理模式。在分布式工作流系统中，正确处理资源文件是保证工作流可靠执行的基础。这一改进使得DolphinScheduler在文件资源管理方面更加稳定可靠。