qoollo / bob

Distributed BLOB storage
MIT License
31 stars 3 forks source link

Properly handle possible `/` at the end of the disk path in `cluster.yaml` #873

Open ikopylov opened 4 months ago

ikopylov commented 4 months ago

Right now regular string concatenation is used to build the path to the alien directory (https://github.com/qoollo/bob/blob/master/bob-backend/src/pearl/settings.rs#L32). This result in double / symbols. Path::join should be used instead.

The same thing happens here: https://github.com/qoollo/bob/blob/master/bob-backend/src/pearl/settings.rs#L267

Probably, there are more places containing similar mistake. All places should be found and fixed