Allow . to match any character, including line separators. about socket.io-client-swift HOT 13 CLOSED

@hanishabsigh can you provide a message that fails to get parsed. You can uncomment https://github.com/socketio/socket.io-client-swift/blob/master/SwiftIO/SocketEngine.swift#L409 to see messages before they get sent to the socket.io-client parser.

from socket.io-client-swift.

I'm actually finding a few problems with regex. Mostly due to the differences between NSString and String.

from socket.io-client-swift.

My object that fails is very large, here is an example I was able to track down:

The following fails:

"42[\"auto\",{\"message_of_the_date\":\"line one\nline two\"}]"

The following succeeds:

"42[\"auto\",{\"message_of_the_date\":\"line one line two\"}]"

The line break in the message of the day causes the parsing of the groups to break.

from socket.io-client-swift.

@nuclearace I just noticed you made some parsing changes, this was tested prior to those changes, thanks

from socket.io-client-swift.

@hanishabsigh It seems to work for me on the latest. Please note #13 though. Since it can cause the regex to fail sometimes. Which is why I changed some things.

from socket.io-client-swift.

@nuclearace This still isn't working for me, incoming messages can't even get passed the following line:

Replacing line 224 with the following worked:

let messageGroups = stringMessage["(\\d*)\\/?(\\w*)?,?(\\d*)?\\[\"(.*?)\",?(.*?)?\\]$", NSRegularExpressionOptions.DotMatchesLineSeparators].groups()

from socket.io-client-swift.

@hanishabsigh try after 44e2781

from socket.io-client-swift.

@nuclearace Checking against the latest commit isn't working, something about the changes in SwiftRegex. So far from what I can tell regex.firstMatchInString in the groups method isn't returning anything. Still investigating. Thanks

from socket.io-client-swift.

This client is able to get this for me:

socket.emit("jsonTest", {foo: "string\ntest test"})

from socket.io-client-swift.

Ok, fixed it by changing the following:

final var targetRange: NSRange {
    return NSRange(location: 0,length: countElements(target))
}

To this:

final var targetRange: NSRange {
    return NSRange(location: 0,length: target.utf16Count)
}

I think the error I was experiencing was happening because my json had variable-width Unicode characters. Found someone having the same issues with swift on StackOverflow: http://stackoverflow.com/questions/25882503/how-can-i-use-nsregularexpression-on-swift-strings-with-variable-width-unicode-c

from socket.io-client-swift.

@hanishabsigh so does c67e7a3 fix it

from socket.io-client-swift.

@nuclearace Yes, that commit fixes it, thanks.

from socket.io-client-swift.

From Xcode 6.3 beta 3 release notes:
>`utf16Count` is removed from String. Instead use count on the UTF16 view of the String. (17627758)

@hanishabsigh @nuclearace

from socket.io-client-swift.