Can't read file using NSData(contentsOfURL: options:)

[ravikiran-Conferra]

I'm using SSZipArchive to unzip the file from the server. Problem is after unzipping i am not able to read the file i am getting the following error The file “UnZippedFile” couldn’t be opened because you don’t have permission to view it(UnZippedFile is the file name given by me).

Here is my code

func saveFetchedDataAtTempDirectory( dataToSave: NSData)
    {
        let zipFilePath = NSTemporaryDirectory().stringByAppendingString("ZipFile")
        let filePath = NSTemporaryDirectory().stringByAppendingString("UnZippedFile")
        do
        {
            try dataToSave.writeToFile(zipFilePath, options: NSDataWritingOptions.DataWritingFileProtectionMask)
            SSZipArchive.unzipFileAtPath(zipFilePath, toDestination: filePath, overwrite: true, password: nil, progressHandler: { (fileName, fileInfo, num1, num2) in
                print("fileName: \(fileName)\n")
                print("info: \(fileInfo)\n")

                }, completionHandler: { (zippedFilePath, done, error) in
                    if done{
                        print("zippedFilePath: \(zippedFilePath)\n")
                        self.fetchJsonFromPath(filePath)
                    }
                    else
                    {
                        print("Error \(error.localizedDescription)")
                    }
            })

        }
        catch let error as NSError
        {
            print("Writing file failed with error\(error.localizedDescription)")
        }
    }

    func fetchJsonFromPath(path: String)
    {
        do
        { // failing here, the error is coming here
            let data = try NSData(contentsOfURL: NSURL(fileURLWithPath: path), options: NSDataReadingOptions.DataReadingMappedAlways)

            do
            {
                let json = try NSJSONSerialization.JSONObjectWithData(data, options: .MutableLeaves)

                if let jsonDict : NSDictionary = json as? NSDictionary
                {
                    print("Json Dict \(jsonDict)")
                }

            } catch
            {
                print("Invalid JSON data received")
            }


        } catch let error as NSError {

            print("Fetching file data failed with error : \(error.localizedDescription)")
        }

    }

[zxl777]

I also encountered this problem, please test.

TEST.ZIP

[jhudsonWA]

Still having an issue? If you are having issues on the NSData() part I'm wondering if you checked to make sure file was at the path. Reopen if still having an issue.

[Coeur]

@jhudsonWA I tested the file from @zxl777 and I reproduced the issue:

Error Domain=NSCocoaErrorDomain Code=257 "The file “017573.log” couldn’t be opened because you don’t have permission to view it." UserInfo={NSFilePath=/Users/coeur/Library/Developer/CoreSimulator/Devices/AAAAAAAA-AAAA-AAAA-AAAA-AAAAAAAAAAAA/data/Containers/Data/Application/AAAAAAAA-AAAA-AAAA-AAAA-AAAAAAAAAAAA/Library/Caches/AAAAAAAA-AAAA-AAAA-AAAA-AAAAAAAAAAAA/017573.log, NSUnderlyingError=0x61800005eba0 {Error Domain=NSPOSIXErrorDomain Code=13 "Permission denied"}}

My code was:

        guard let unzipPath = tempUnzipPath() else {
            return
        }
        _ = SSZipArchive.unzipFile(atPath: zipPath, toDestination: unzipPath)
        let items = try! FileManager.default.contentsOfDirectory(atPath: unzipPath)
        do {
            let data = try Data(contentsOf: URL(fileURLWithPath: unzipPath + "/" + items[0]), options: .alwaysMapped)
            print("success: \(data.count)")
        } catch {
            print(error)
        }

[Coeur]

OK, apparently @zxl777 files inside the archive have no read permission, and our unzipFileAtPath:toDestination: is respecting those permissions. If I use a different tool to uncompress it, like Finder, then permissions are set to read+write.

We respect the original permissions because of SSZipArchive.m line 418:

                            // Set the original permissions on the file
                            uLong permissions = fileInfo.external_fa >> 16;
                            if (permissions != 0) {
                                // Store it into a NSNumber
                                NSNumber *permissionsValue = @(permissions);

                                // Retrieve any existing attributes
                                NSMutableDictionary *attrs = [[NSMutableDictionary alloc] initWithDictionary:[fileManager attributesOfItemAtPath:fullPath error:nil]];

                                // Set the value in the attributes dict
                                attrs[NSFilePosixPermissions] = permissionsValue;

                                // Update attributes
                                if ([fileManager setAttributes:attrs ofItemAtPath:fullPath error:nil] == NO) {
                                    // Unable to set the permissions attribute
                                    NSLog(@"[SSZipArchive] Failed to set attributes - whilst setting permissions");
                                }

#if !__has_feature(objc_arc)
                                [attrs release];
#endif
                            }

[Coeur]

@jhudsonWA maybe we could provide an option that will set all uncompressed files to read+write, like Archive Utility is doing? See #348

[Coeur]

@ravikiran-Conferra @zxl777 I successfully workaround the permissions with:

        do {
            // Retrieve any existing attributes
            var attrs = try FileManager.default.attributesOfItem(atPath: fullPath)
            let existing = (attrs as NSDictionary).filePosixPermissions()
            // Set the read+write value in the attributes dict
            attrs[.posixPermissions] = existing | 0b110000000
            // Update attributes
            try FileManager.default.setAttributes(attrs, ofItemAtPath: fullPath)
            
            let data = try Data(contentsOf: URL(fileURLWithPath: fullPath, isDirectory: false), options: .alwaysMapped)
            print("success: \(data.count)")
        } catch {
            print(error)
        }

I've also post it as a StackOverflow answer on https://stackoverflow.com/a/45048822/1033581

[jhudsonWA]

@Coeur I like the idea of providing an option/override with the default add read+write, lets add it for 2.0