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

问题背景

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

问题现象

当用户通过DolphinScheduler的Web界面重新上传文件时，系统会错误地抛出"文件扩展名不能更改"的异常，即使实际上用户并没有修改文件扩展名。这种错误提示会阻止用户正常完成文件更新操作。

技术分析

经过深入分析，发现问题出在文件扩展名校验的代码逻辑上。原始代码在获取上传文件的文件名时，没有正确处理MultipartFile对象的文件名获取方式。

在Spring框架中，上传的文件被封装为MultipartFile对象，获取文件名应该使用getOriginalFilename()方法，而不是getName()方法。getName()方法返回的是表单中文件字段的名称，而不是实际文件的名称，这导致了错误的扩展名比较结果。

解决方案

正确的实现方式应该优先使用getOriginalFilename()获取原始文件名，当该方法返回null时才回退到getName()方法。修改后的校验逻辑如下：

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

问题影响

这个bug会影响所有需要更新资源文件的用户场景，特别是：

1. 需要替换现有文件的用户
2. 需要上传同名但内容更新的文件的用户
3. 需要维护资源库中文件的用户

最佳实践建议

对于文件上传功能的实现，建议开发者：

1. 始终明确区分文件字段名和实际文件名
2. 对文件扩展名校验提供更友好的错误提示
3. 考虑在UI层面提前验证文件扩展名
4. 记录详细的文件上传日志，便于问题排查

总结

文件上传是工作流管理系统中的基础但重要的功能。Apache DolphinScheduler通过修复这个文件扩展名校验的问题，提升了用户体验和系统可靠性。开发者在使用MultipartFile时应当注意其API的细节差异，避免类似的实现错误。