77

I'm trying to upload an image with parameters in Swift. When I try this code, I can get the parameters but not the image

uploadFileToUrl(fotiño:UIImage){
    var foto =  UIImage(data: UIImageJPEGRepresentation(fotiño, 0.2))


    var request = NSMutableURLRequest(URL:NSURL(string: "URL"))
    request.HTTPMethod = "POST"

    var bodyData = "id_user="PARAMETERS&ETC""


    request.HTTPBody = bodyData.dataUsingEncoding(NSUTF8StringEncoding);
    request.HTTPBody = NSData.dataWithData(UIImagePNGRepresentation(foto))
    println("miraqui \(request.debugDescription)")
    var response: AutoreleasingUnsafeMutablePointer<NSURLResponse?>=nil
    var HTTPError: NSError? = nil
    var JSONError: NSError? = nil

    var dataVal: NSData? =  NSURLConnection.sendSynchronousRequest(request, returningResponse: response, error: &HTTPError)

    if ((dataVal != nil) && (HTTPError == nil)) {
        var jsonResult = NSJSONSerialization.JSONObjectWithData(dataVal!, options: NSJSONReadingOptions.MutableContainers, error: &JSONError)

        if (JSONError != nil) {
            println("Bad JSON")
        } else {
            println("Synchronous\(jsonResult)")
        }
    } else if (HTTPError != nil) {
        println("Request failed")
    } else {
        println("No Data returned")
    }
}

edit 2:

I think that I have some problems with the path of the saved UIImage, because php tells me that the file already exist, which I think is because I send it in blank

func createRequest (#userid: String, disco: String, id_disco: String, pub: String, foto: UIImage) -> NSURLRequest {
    let param = [
        "id_user"  : userid,
        "name_discoteca"    : disco,
        "id_discoteca" : id_disco,
        "ispublic" : pub] // build your dictionary however appropriate

    let boundary = generateBoundaryString()

    let url = NSURL(string: "http....")
    let request = NSMutableURLRequest(URL: url)
    request.HTTPMethod = "POST"
    request.timeoutInterval = 60
    request.HTTPShouldHandleCookies = false
    request.setValue("multipart/form-data; boundary=\(boundary)", forHTTPHeaderField: "Content-Type")
    var imagesaver = ImageSaver()

    var image = foto  // However you create/get a UIImage
    let documentsPath = NSSearchPathForDirectoriesInDomains(.DocumentDirectory, .UserDomainMask, true)[0] as String
    let destinationPath = documentsPath.stringByAppendingPathComponent("VipKing.jpg")
    UIImageJPEGRepresentation(image,1.0).writeToFile(destinationPath, atomically: true)


    self.saveImage(foto, withFileName: "asdasd22.jpg")


    var path = self.documentsPathForFileName("asdasd22.jpg")


    self.ViewImage.image = self.loadImageWithFileName("asdasd22.jpg")



  //  let path1 = NSBundle.mainBundle().pathForResource("asdasd22", ofType: "jpg", inDirectory: path) as String! 

    **//path1 always crash**


    println(param.debugDescription)
    println(path.debugDescription)
    println(boundary.debugDescription)




    request.HTTPBody = createBodyWithParameters(param, filePathKey: "asdasd22.jpg", paths: [path], boundary: boundary)

    println(request.debugDescription)


    return request
}
Jojodmo
  • 23,357
  • 13
  • 65
  • 107
Pedro Manfredi
  • 3,768
  • 5
  • 19
  • 24
  • What was your problem with `filePathKey` i am facing the same problem.? – Amr Mohamed Apr 13 '15 at 20:47
  • Upload image simply...[Using Alamofire](http://stackoverflow.com/questions/32366021/alamofire-error-code-1000-the-url-does-not-point-to-a-file-url/36863066#36863066) – kalpesh jetani Apr 26 '16 at 12:01

3 Answers3

148

In your comment below, you inform us that you are using the $_FILES syntax to retrieve the files. That means that you want to create a multipart/form-data request. The process is basically:

  1. Specify a boundary for your multipart/form-data request.

  2. Specify a Content-Type of the request that specifies that it multipart/form-data and what the boundary is.

  3. Create body of request, separating the individual components (each of the posted values as well as between each upload).

For more detail, see RFC 7578. Anyway, in Swift 3 and later, this might look like:

/// Create request
///
/// - parameter userid:   The userid to be passed to web service
/// - parameter password: The password to be passed to web service
/// - parameter email:    The email address to be passed to web service
///
/// - returns:            The `URLRequest` that was created

func createRequest(userid: String, password: String, email: String) throws -> URLRequest {
    let parameters = [
        "user_id"  : userid,
        "email"    : email,
        "password" : password]  // build your dictionary however appropriate
    
    let boundary = generateBoundaryString()
    
    let url = URL(string: "https://example.com/imageupload.php")!
    var request = URLRequest(url: url)
    request.httpMethod = "POST"
    request.setValue("multipart/form-data; boundary=\(boundary)", forHTTPHeaderField: "Content-Type")
    
    let fileURL = Bundle.main.url(forResource: "image1", withExtension: "png")!
    request.httpBody = try createBody(with: parameters, filePathKey: "file", urls: [fileURL], boundary: boundary)
    
    return request
}

/// Create body of the `multipart/form-data` request
///
/// - parameter parameters:   The optional dictionary containing keys and values to be passed to web service.
/// - parameter filePathKey:  The optional field name to be used when uploading files. If you supply paths, you must supply filePathKey, too.
/// - parameter urls:         The optional array of file URLs of the files to be uploaded.
/// - parameter boundary:     The `multipart/form-data` boundary.
///
/// - returns:                The `Data` of the body of the request.

private func createBody(with parameters: [String: String]? = nil, filePathKey: String, urls: [URL], boundary: String) throws -> Data {
    var body = Data()
    
    parameters?.forEach { (key, value) in
        body.append("--\(boundary)\r\n")
        body.append("Content-Disposition: form-data; name=\"\(key)\"\r\n\r\n")
        body.append("\(value)\r\n")
    }
    
    for url in urls {
        let filename = url.lastPathComponent
        let data = try Data(contentsOf: url)
        
        body.append("--\(boundary)\r\n")
        body.append("Content-Disposition: form-data; name=\"\(filePathKey)\"; filename=\"\(filename)\"\r\n")
        body.append("Content-Type: \(url.mimeType)\r\n\r\n")
        body.append(data)
        body.append("\r\n")
    }
    
    body.append("--\(boundary)--\r\n")
    return body
}

/// Create boundary string for multipart/form-data request
///
/// - returns:            The boundary string that consists of "Boundary-" followed by a UUID string.

private func generateBoundaryString() -> String {
    return "Boundary-\(UUID().uuidString)"
}

With:

extension URL {
    /// Mime type for the URL
    ///
    /// Requires `import UniformTypeIdentifiers` for iOS 14 solution.
    /// Requires `import MobileCoreServices` for pre-iOS 14 solution

    var mimeType: String {
        if #available(iOS 14.0, *) {
            return UTType(filenameExtension: pathExtension)?.preferredMIMEType ?? "application/octet-stream"
        } else {
            guard
                let identifier = UTTypeCreatePreferredIdentifierForTag(kUTTagClassFilenameExtension, pathExtension as CFString, nil)?.takeRetainedValue(),
                let mimeType = UTTypeCopyPreferredTagWithClass(identifier, kUTTagClassMIMEType)?.takeRetainedValue() as String?
            else {
                return "application/octet-stream"
            }

            return mimeType
        }
    }
}

extension Data {
    
    /// Append string to Data
    ///
    /// Rather than littering my code with calls to `data(using: .utf8)` to convert `String` values to `Data`, this wraps it in a nice convenient little extension to Data. This defaults to converting using UTF-8.
    ///
    /// - parameter string:       The string to be added to the `Data`.
    
    mutating func append(_ string: String, using encoding: String.Encoding = .utf8) {
        if let data = string.data(using: encoding) {
            append(data)
        }
    }
}

Having all of this, you now need to submit this request. I would advise this is done asynchronously. For example, using URLSession, you would do something like:

let request: URLRequest

do {
    request = try createRequest(userid: userid, password: password, email: email)
} catch {
    print(error)
    return
}

let task = URLSession.shared.dataTask(with: request) { data, response, error in
    guard let data = data, error == nil else {
        // handle error here
        print(error ?? "Unknown error")
        return
    }
    
    // parse `data` here, then parse it
    
    // note, if you want to update the UI, make sure to dispatch that to the main queue, e.g.:
    //
    // DispatchQueue.main.async {
    //     // update your UI and model objects here
    // }
}
task.resume()

If you are uploading large assets (e.g. videos or the like), you might want to use a file-based permutation of the above. See https://stackoverflow.com/a/70552269/1271826.


For Swift 2 renditions, see previous revision of this answer.

Rob
  • 415,655
  • 72
  • 787
  • 1,044
  • I try a lot of things saving in local the image... let documentsPath = NSSearchPathForDirectoriesInDomains(.DocumentDirectory, .UserDomainMask, true)[0] as String i could save it, but the php tell me than already exist when i try to upload it, i think than i send the image in blank or smth goes wrong... i think i'm confuse between paths and filePathKey i edit my answer! – Pedro Manfredi Oct 05 '14 at 03:36
  • @user3426109 By the way, the choice of `NSSearchPathForDirectories` or `NSBundle` is how you specify which local file you're uploading. But, if you're getting your "already exist" message from the server, then this has nothing to do with that. The "already exist" indicates that the _server_ has a copy of a file of that name (and apparently the script is written such that it won't just overwrite it). – Rob Oct 05 '14 at 22:01
  • Done!!! Thank you so much!!!!!!! it was on filePathKey: "file" where i have the problem – Pedro Manfredi Oct 06 '14 at 01:27
  • Very nice! I wanted to switch to a swift version of this and thought I would look for a quick solution. Found it here! Im doing this to post to server thats also taking in form submissions and requires multiple files... so file[] instead of file for the filePathKey was needed. Thanks again. – bworby Jan 15 '15 at 05:11
  • how is this done when you are passing a file in a post parameter? I can't seem to figure it out http://stackoverflow.com/questions/30006290/uploading-image-in-swift-with-multiple-parameters – Tyler May 02 '15 at 19:52
  • It would be great to get the code updated to Swift 2 (the `mimeTypeForPath` part isn't quite straightforward to me...) – Nicolas Miari Dec 22 '15 at 06:23
  • 4
    @NicolasMiari - This is Swift 2. As the comment for `mimeTypeForPath` says, you need `MobileCoreServices`. So, up at the top of the file, do `import MobileCoreServices`. – Rob Dec 22 '15 at 08:15
  • I am not able to find `appendString` method under `NSMutableData`? – Tapas Pal Feb 08 '16 at 09:56
  • @TapasPal See the `extension` for `NSMutableData` which I included in my code samples above. – Rob Feb 08 '16 at 10:01
  • How to upload multiple images in an array ? Any idea at all. – G.Abhisek Mar 28 '16 at 08:41
  • Hi @Rob quick question, would this work for what I am trying to do here http://stackoverflow.com/questions/40527140/mysql-and-swift-upload-image-would-it-be-better-to-use-afnetworking Hope you don't mind me asking! –  Nov 20 '16 at 22:58
  • Hey @Rob, just a quick question. In your example you are loading the file into memeory `let data = try Data(contentsOf: url)`. If we are handling files with a considerable size, this would not be nice. Do you know if there's a way to do it using an InputStream or something like that? – Tiago Pereira Jun 06 '17 at 17:52
  • Yeah, both stream-based and file-based approaches can be used to minimize memory impact. And, as an addition wrinkle, if they’re really big, you might consider using background uploads, which must be file-based. So if I were uploading huge assets, I’d use stream based API to build file with the body of the request (minimizing peak memory usage when building what will be the body of the request) and then do file-based upload task which keeps memory usage during upload to a minimum, and then remove that file when the upload is done. – Rob Jun 06 '17 at 17:59
  • Hi @Rob, ATM I'm currently uploading images directly to AWS S3 from the mobile application, retrieving the url of the images, then sending another request and uploading the urls to my backend (MySql/PHP) and storing the urls. Would you recommend against this and use the code you posted to upload the images to the backend and then let the backend to upload the images to s3? – luke Jun 10 '17 at 14:37
  • 1
    @luke - It’s a matter of opinion, but it feels like the client app is a bit entangled in implementation details of the back end architecture. You’re also likely dealing with two authentication systems and two points of failure. I’d probably lean towards a single end point for the app, and have the web service manage the image storage. – Rob Jun 10 '17 at 16:26
  • I'm getting an error : "JSON text did not start with array or object and option to allow fragments not set." – Krutika Sonawala Jan 12 '18 at 06:03
  • 1
    @KrutikaSonawala - Above, I assumed you had a web service that wrote JSON responses. (It's one of the most robust ways to have a web service return a parseable response to a client app.) Perhaps your web service isn't designed to return a JSON response. Or perhaps you had an error in your web service that prevented it from creating a proper JSON response. I can't diagnose that on the basis of your comment. If the aforementioned isn't clear, I'd suggest you post your own question showing us what you did, what you expected in response, and what you actually received in response. – Rob Jan 12 '18 at 06:19
  • @Rob here's the link of my question please. https://stackoverflow.com/questions/48220802/alamofire-file-upload-getting-error-json-text-did-not-start-with-array-or-objec – Krutika Sonawala Jan 12 '18 at 06:36
  • Thanks you for that high quality answer, makes dev lives much easier but also teaches how to do things. TipTop! – Peter Pohlmann Aug 26 '20 at 09:31
  • How can I send URL as a file name? or can we send fileName and URL as another parameter? – Lakshmi Yadav Oct 09 '20 at 08:21
  • I’d personally be inclined to send it as a separate parameter. – Rob Oct 09 '20 at 14:50
14

AlamoFire now supports Multipart:

https://github.com/Alamofire/Alamofire#uploading-multipartformdata

Here's a blog post with sample project that touches on using Multipart with AlamoFire.

http://www.thorntech.com/2015/07/4-essential-swift-networking-tools-for-working-with-rest-apis/

The relevant code might look something like this (assuming you're using AlamoFire and SwiftyJSON):

func createMultipart(image: UIImage, callback: Bool -> Void){
    // use SwiftyJSON to convert a dictionary to JSON
    var parameterJSON = JSON([
        "id_user": "test"
    ])
    // JSON stringify
    let parameterString = parameterJSON.rawString(encoding: NSUTF8StringEncoding, options: nil)
    let jsonParameterData = parameterString!.dataUsingEncoding(NSUTF8StringEncoding, allowLossyConversion: true)
    // convert image to binary
    let imageData = UIImageJPEGRepresentation(image, 0.7)
    // upload is part of AlamoFire
    upload(
        .POST,
        URLString: "http://httpbin.org/post",
        multipartFormData: { multipartFormData in
            // fileData: puts it in "files"
            multipartFormData.appendBodyPart(fileData: jsonParameterData!, name: "goesIntoFile", fileName: "json.txt", mimeType: "application/json")
            multipartFormData.appendBodyPart(fileData: imageData, name: "file", fileName: "iosFile.jpg", mimeType: "image/jpg")
            // data: puts it in "form"
            multipartFormData.appendBodyPart(data: jsonParameterData!, name: "goesIntoForm")
        },
        encodingCompletion: { encodingResult in
            switch encodingResult {
            case .Success(let upload, _, _):
                upload.responseJSON { request, response, data, error in
                    let json = JSON(data!)
                    println("json:: \(json)")
                    callback(true)
                }
            case .Failure(let encodingError):
                callback(false)
            }
        }
    )
}

let fotoImage = UIImage(named: "foto")
    createMultipart(fotoImage!, callback: { success in
    if success { }
})
Robert Chen
  • 5,179
  • 3
  • 34
  • 21
2

Thank you @Rob, your code is working fine, but in my case, I am retriving image from gallary and taking name of the image by using code:

let filename = url.lastPathComponent

But this code, displaying image extension as .JPG (in capital letter), but server not accepting extensions in captital letter, so i changed my code as:

 let filename =  (path.lastPathComponent as NSString).lowercaseString

and now my code is working fine.

Thank you :)

itsji10dra
  • 4,603
  • 3
  • 39
  • 59