chore: merge release into master #4298
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
In case you open Chrome DevTools and close them while the iOS runtime still sends messages, an error is raised in CLI: ``` Error: WebSocket is not open: readyState 2 (CLOSING) ``` The problem is that we are using Transform stream between the communication and close event of the socket is not raised until the stream flushes its current state. Fix it by skipping the write operation in case the socket is not in OPEN state.
Update ios-device-lib to 0.5.0, so we can use the new fixes and features: https://github.com/telerik/ios-device-lib/releases/tag/v0.5.0
During debug operation, iOS Runtime "talks" to Chrome DevTools. CLI stands as proxy between them and translates the messages. Due to Node.js socket implementation, some messages may come on chunks. The format of each message is: - first 4 bytes contain the lenght of the message (in Big-Endian) - the next bytes are the actual message Currently, the following behavior is observed: - when a full message is received, CLI sends it correctly to Chrome DevTools - when more than one message is received in the data, CLI sends only the first message to Chrome DevTools and disregards the other part - in case a split message is received, when the first part is only part of the first 4 bytes (part of the length), CLI fails with `Index out of Range` or `ERR_BUFFER_OUT_OF_BOUNDS` error (based on the inner state of the variables). To fix this behavior, remove CLI's parsing and reuse the one from `ios-device-lib`.
…f-range-debug fix: errors during debug on iOS
In case you use `tns run ios --path <path to app> --device <simulator GUID>`, CLI will try to stop the application, but may kill CLI instead. The problem is in ios-sim-portable, which incorrectly searches for application PID. Update the package to 4.0.7 where the issue is fixed.
fix: trying to stop app on simulator may kill CLI
On slower mac machines, an error may be raised during restart of app on iOS Simulator. The problem is that we stop the application and try to start it after that. However, the stop method may be resolved before the application is actually killed. Trying to start it after that leads to undefined behavior and errors in some cases. To fix this, update ios-sim-portable to 4.0.8 where a possible fix is applied.
…p-app fix: error may be raised during restart of app on ios Sim
docs: update CHANGELOG for 5.1.1
DimitarTachev
approved these changes
Jan 18, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Checklist
What is the current behavior?
What is the new behavior?
Fixes/Implements/Closes #[Issue Number].