1. customerDTO.getMessageDtoList().get(0).getAttachments().get(0).getFileName() - это кандидат на NPE
2. filesMap судя по цвету это переменная класса, если текущий класс является сервисом (считай Синглтон) то у вас потенциальные проблемы многопоточностью
3. зачем тут нужне filesMap.computeIfPresent(...)? переменные должны нести смысл, от arg1, arg2 толку мало.
4. log.error() зачем нужен? Судля по тексту ошибки ты либо должне кидать исключение, либо просто игнорировать.