Dragged mp3 file gets renamed as "foo.mp3_.mp3"

[timbl]

Eg https://timbl.com/timbl/Public/Test/Music/Les%20Hayden/Proverbs/Les_Hayden_-_01_-_Gift_Horse.mp3_.mp3

The algorithm should notice that the mime type of the file is fine as it .. and not add another extension.

[timbl]

That file was made by dragging a file from the desktop onto the green plus of the folder.

(Free music for testing from the Free Music Archive https://freemusicarchive.org/music/Les_Hayden/Proverbs_1720 )

[timbl]

hmm

> mime.lookup('foo.mp3')
'audio/mpeg'
> mime.extension('audio/mp3')
'mp3'
> mime.extension('audio/mpeg')
'mpga'
> mime.lookup('foo.mpga')
'audio/mpeg'
> 

[bourgeoa]

I don't know what is your NSS version.

I tried to drag a .mp3 file to a pod with NSSv5.2.0 and the file on the filesystem appears to be :
file.mp3_.mp3$.mp3
I think it indicates that the solution can only be a combination of solid-ui that adds _.mp3 and NSS that on top adds $.mp3.

• I think that the check if contentType is correct for the file extension should not be done in 2 different places
  If done in NSS it should be deleted in solid-ui
• the check in NSSv5.2.0 will display in solid-ui file.mp3 and on the filesystem file.mp3$.mp3
  that is not satisfactory. I think that in such cases the file in the filesystem should be file.mp3 and in general 'file.ext$.ext' should always be 'file.ext'

(I don't know if there is an other case than mp3 in mime-types)

[RubenVerborgh]

NSS issue is separate (but same cause) and fixed in nodeSolidServer/node-solid-server#1328

[bourgeoa]

https://github.com/solid/solid-ui/pull/147 solves the point suppressing double check in solid-ui and NSS

[megoth]

@bourgeoa I've tested the latest version of dev on node-solid-server (which has nodeSolidServer/node-solid-server#1336 merged into it), and it seems to me that we don't need a fix for solid-ui. (Your fix being https://github.com/solid/solid-ui/pull/147.)

@timbl I suspect this to be the case for your PR as well (https://github.com/solid/solid-ui/pull/144).

With the fix in NSS, when I drag and drop a mp3 file onto the green plus icon, the resulting filename for file_example_MP3_700KB.mp3 is still file_example_MP3_700KB.mp3 (not file_example_MP3_700KB.mp3_.mp3 which happened earlier).

Am I missing something, or could we close these two PRs?

[timbl]

I think you are missing something. I am not sure why yours works without #144. The problem is for any mime type extension where mapping is not symmetrical, the old code will mistakenly think that the extension is wrong when in fact it is right. If your case works, then do you still have at the console these results?

> mime.lookup('foo.mp3')
'audio/mpeg'
> mime.extension('audio/mp3')
'mp3'
> mime.extension('audio/mpeg')
'mpga'
> mime.lookup('foo.mpga')
'audio/mpeg'
> 

The extension must be allowed if EITHER it is the right extsnion for the mime type OR th emime type is the right one for the extension. Maybe when there > 1 extsnsions, then the module gives randomly each one... so it will fail randomly? Not sure why yours worked., Not a function of NSS at all.

[RubenVerborgh]

Not a function of NSS at all.

@timbl It depends.

In NSS 4.x, a file that was POST/PUT to the server would need to have the correct extension, or NSS would think it was Turtle.

In NSS 5.x, only the content type matters.

So why is Mashlib adding this extension?

[bourgeoa]

https://github.com/solid/solid-ui/pull/147 solves the point suppressing double check in solid-ui and NSS