Broken shareFilesFromDirectory() ?

[kimaldis]

I was having trouble serving files from a location using shareFilesFromDirectory and tracked it down to a couple of lines in Socket+File.swift. This may be completely the wrong solution and it seems odd that no-one's picked up on it but it's working for me now so if it helps anyone ...

line 10:
#if os(iOS) || os(tvOS) || os (Linux) || os (macOS) // ka added macOS

and line 44:
#if os(iOS) || os(tvOS) || os (Linux) || os (macOS) // KA: Added macOS

[Joebayld]

This broke for me as well on the latest version... 1.4.6.

However, I tried your fix and it didn't work.

[Joebayld]

Turns out the bug was introduced in this commit... perhaps we need better testing on here. 18d34f2

[Vkt0r]

Hey, @Joebayld @kimaldis Thanks for reporting this. I would need more details about the issue, can you provide with a sample project with the reproduced bug?

[kimaldis]

I've dropped a stripped down version of the project into my dropbox here . sorry about any junk left in there.

built out of the box it fails when you browse to http://localhost:8082/root/<foo.html. Editing the two lines I've marked in Socket+File.swift to include (macOS) should fix it. I found I had to clean the build folder to get Socket+File.swift to compile

You'll need a folder with some html in an NSHomeDirectory ( "/Users//Library/Containers/com.kim-aldis.EarwigServer/Data/htdocs")

[kimaldis]

macOS Mojave, 10.14.4, Xcode Version 10.2 (10E125), by the way

[Vkt0r]

@kimaldis Thanks for the sample project! I'll take a look as soon as I can.

[Joebayld]

How goes this? Need any help? Perhaps we make a release without the changes made to 'shareFilesFromDirectory()' as this is a breaking change?

[Vkt0r]

@kimaldis I found the source of your error! Your error was coming from the impossibility of the sendfile function to write into the socket, see below:

swifter/XCode/Sources/Socket+File.swift

Lines 39 to 53 in 3367b63

	public func writeFile(_ file: String.File) throws {
	var offset: off_t = 0
	var sf: sf_hdtr = sf_hdtr()
	#if os(iOS) || os(tvOS) || os (Linux)
	let result = sendfileImpl(file.pointer, self.socketFileDescriptor, 0, &offset, &sf, 0)
	#else
	let result = sendfile(fileno(file.pointer), self.socketFileDescriptor, 0, &offset, &sf, 0)
	#endif
	if result == -1 {
	throw SocketError.writeFailed("sendfile: " + Errno.description())
	}
	}
	}

If you put a breakpoint in line 50 you will see the error being returned for the function and it's

Operation not permitted.

So I started to investigate a little and I noticed this in this thread:

Sandboxed apps cannot read files from outside the sandbox, except where explicitly given permission by the user via an Open File dialog, which returns a "security scoped URL".

When I went to your app Capabilities you had the Sandbox enabled, so once I disabled it the app started to work fine.

Also in the DemoServer you can test that in the stable branch the shareFilesFromDirectory() it's working perfectly if you try to create files inside the directory inside the SwiterSampleOSX.

PD: I tested your demo project using the latest changes in the stable branch.

[Vkt0r]

@Joebayld I don't know exactly what are you doing and how but as I explained above I tested the shareFilesFromDirectory() in the DemoServer.swift and it's working perfectly. So please check it out your problem or try to create sample project with the reproduced bug in order to help you.

[Joebayld]

I'm sharing files that are inside my App Bundle via shareFilesFromDirectory(Bundle.main.resourcePath! + "/dist/assets")..

I had sandbox disabled and it still didn't work. I can try to make a test project - unless you just try sharing a file from inside the app bundle on your test project?

[Joebayld]

Update - still not working for me in 1.4.6. Below is how I'm using the server.. the index.html file references static assets. I constantly get a error 404 not found in version 1.4.6.

[Vkt0r]

@Joebayld I can confirm that's working fine as in the DemoServer.swift when you run the SwifterSampleOSX macOS app the dir is the one in the main bundle. But you can replace it with your line and it should be there a logo.png we have in the app.

Change this:

let server = demoServer(try String.File.currentWorkingDirectory())

to this in the main.swift:

let server = demoServer(Bundle.main.resourcePath!)

And run the sample project. Then you can try to get the file in the address http://localhost:9080/public/logo.png. Also, I created another file myself in the path and it's working fine.

What I recommend you is after you get the Podfile pointing to stable clear your Derived Data dir and clean and build your project again.

If that's doesn't solve your problem I recommend you put a breakpoint in the shareFilesFromDirectory and see what's happening with your file there or create a sample project in order to see the issue closely.