背景

用户头像上传性能中,服务端上传流程如下:

  1. base64解码成字符串
  2. 图片写入服务器本地长期目录
  3. 上传到阿里云OSS/七牛云存储
  4. 图片审核

代码如下

type Parms struct {    Head string //客户端发送base64字符串}func Upload(p Params){    if p.Head == ""{        return errors.New("img is empty")    }        str,err := Base64Decode(p.Head)    if err != nil{        return err    }        filename := makeUniqueName()    if err := FileWrite(filename,str);err != nil{         return err    }        err := UploadAliOss(filename)    if err != nil{        return err    }        err := VerifyImg(filename)    if err != nil{        return err    }}

问题

以前多个我的项目通过jenkins公布不同的上线指令,每新建一个我的项目,运维须要独自写一套shell命令,无奈复用之前的,保护老本比拟高。起初咱们降级部署形式,只须要批改我的项目部署目录配置文件,具体后续再介绍。

始终以来稳固运行,周末收到反馈呈现问题,用户无奈上传头像。

  • 查看代码的git历史提交记录,近期没有批改代码,排除代码bug
  • 报错提醒只在调用FileWriteUploadAliOss两个办法时呈现
  • 在打包镜像的时候没有在容器中创立长期目录,所以无奈写入文件,用户头像无奈上传

长期解决

手动在容器中创立一个长期目录,保留图片文件。

齐全解决

我认为,图片文件全副保留在第三方对象存储中,服务器保留一份是齐全没必要的,只会占用磁盘空间。

咱们能够创立Reader对象把图片文件读取到缓冲区,再上传到第三方对象存储。

代码如下

func Upload(p Params){    if p.Head == ""{        return errors.New("img is empty")    }        str,err := Base64Decode(p.Head)    if err != nil{        return err    }        filename := makeUniqueName()    buf := bytes.NewReader([]byte(str))        err := UploadAliOssBuff(filename,buf)    if err != nil{        return err    }        err := VerifyImg(filename)    if err != nil{        return err    }}

总结

在接手保护老我的项目时候,防止不了遇到坑。除了吐槽前人写的蹩脚外,咱们能够尽我所能晋升代码保护品质,让后来者更容易保护。

一片空地,可能开始只有几片垃圾。如果始终不清理,你来丢,他也丢,很快就会变成垃圾堆。

微信公众号