Server: actually return MIME type

[MinchinWeb]

The original paths being provided had a tailing slash, and so everything was returning as "application/octet-stream" (at least on Windows)

Pull Request Checklist

Resolves: #2880

Refer to that issue for the discussion.

• Ensured tests pass and (if applicable) updated functional test output
• Conformed to code style guidelines by running appropriate linting tools
• [ n/a ] Added tests for changed code
• [ n/a ] Updated documentation for changed code

P.S. If this could be marked "hacktoberfest-accepted" (so it counts towards my Hacktoberfest Pull Requests), that would be appreciated!

[MinchinWeb]

@avaris The issue here seems to have been fixed between the 4.7.0 release and now; I suspect by #2931 (because that's the only thing that's touched the server module). As it stands right now, the master branch is returning the MIME types as expected.

So either close this as no longer needed, or maybe these changes are still helpful anyway.

[avaris]

I suppose we should close this. I'm still curious where that extra / came from, though. If #2931 helped this issue, I'm slightly puzzled. Does your python return True for os.path.exists("file.css/") provided that there is a file.css in the same folder? It should not, normally...

[MinchinWeb]

I'm not really sure what was causing the issues... os.path.exists("file.css/") seems to return False (as expected). I think it was mistakenly adding the / to paths it wasn't supposed to, but I could never catch it in action, which is why this PR worked by pulling it off when trying to determine MIME types.

I'll close this.