fix: Allow disabling file name sanitization #1586
Conversation
taratatach
force-pushed
the
feat/add-file-creation-option-to-disable-sanitization
branch
2 times, most recently
from
194c053 to
c35efeb
Compare
| * @returns {Promise<object>} - A promise that returns the copied file if resolved. | ||
| * @throws {FetchError} | ||
| * | ||
| */ | ||
| async copy(id, name, dirId) { | ||
| async copy(id, name, dirId, { sanitizeName = true } = {}) { |
There was a problem hiding this comment.
An alternative would have been to pass a function rather than a bool, like:
{ sanitizeFct = sanitizeAndValidateFileName } = {}
And then:
if (sanitizeFct) { sanitizeFct(name) }
Not a big deal but I find it a bit more elegant than using a boolean
There was a problem hiding this comment.
It would open up some use cases but I don't find copy(id, name, dirId, { sanitizeFct: undefined }) that elegant.
There was a problem hiding this comment.
Maybe sanitizeFct: () => {} would be better ? 😄
Anyways, I'm ok with both solutions, it's a matter of preference, so not a requirement to me
taratatach
force-pushed
the
feat/add-file-creation-option-to-disable-sanitization
branch
from
c35efeb to
89f840f
Compare
Some clients, like Cozy Desktop, do their own file name sanitization and don't expect `cozy-client` to do it for them. In fact, this sanitization would lead to issues down the road. To prevent these, we add a `sanitizeName` option to `FileCollection`'s methods to disable `cozy-client`'s sanitization. The option's value will be `true` by default so other clients don't have anything to do.
taratatach
force-pushed
the
feat/add-file-creation-option-to-disable-sanitization
branch
from
89f840f to
454d28b
Compare
| async copy(id, name, dirId) { | ||
| async copy(id, name, dirId, { sanitizeName = true } = {}) { | ||
| let sanitizedName = name | ||
| if (sanitizedName != null && sanitizeName) { |
There was a problem hiding this comment.
You slightly changed the condition here, from === undefined to != null. Is there a reason ?
There was a problem hiding this comment.
This is a habit I have because usually, when we don't want undefined, we don't want null either. And in this case, sanitizing null would throw TypeError: Cannot read properties of null (reading 'trim').
taratatach
deleted the
feat/add-file-creation-option-to-disable-sanitization
branch
Some clients, like Cozy Desktop, do their own file name sanitization
and don't expect
cozy-clientto do it for them. In fact, thissanitization would lead to issues down the road.
To prevent these, we add a
sanitizeNameoption toFileCollection'smethods to disable
cozy-client's sanitization.The option's value will be
trueby default so other clients don'thave anything to do.