This repository has been archived by the owner on Dec 29, 2020. It is now read-only.
REM-25242 Fix redirect handling in custom protocol #15
Conversation
This file contains 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
- inform the protocol client when there is a valid redirect request - remove tracking of the session data task from the custom protocol because it is already tracked in the swizzle method _rpt_setState: - either use UIWebView swizzling or the custom protocol to track UIWebView requests, not both
…wizzling and protocol approaches for request tracking are mutually exclusive
Codecov Report
@@ Coverage Diff @@
## master #15 +/- ##
=========================================
Coverage ? 80.31%
=========================================
Files ? 30
Lines ? 1214
Branches ? 0
=========================================
Hits ? 975
Misses ? 239
Partials ? 0
Continue to review full report at Codecov.
|
donnie-jp
pushed a commit
to donnie-jp/ios-perftracking
that referenced
this pull request
Dec 14, 2018
…~DONNIE.SMITH/perftracking_ds:bugfix/rem-20877-uicontroltests-queuefix to master * commit '024c92636056601d23b36af57c55bb4b88f0c0fe': REM-20877 Add a flag `forceTrackingEnabled` to the TrackingManager API REM-20877 Move HostApp into Tests folder
donnie-jp
pushed a commit
to donnie-jp/ios-perftracking
that referenced
this pull request
Dec 14, 2018
…TH/perftracking_ds:release/1.1-merge-from-gh-20180130 to master * commit 'dcfd3ee904abb3f238e296aea80297d9e5e7f95e': REM-25612 bump podspec for 1.1 release (rakutentech#25) feat(swizzling): REM-25021 Rewrite NSURLSessionTask swizzling (rakutentech#24) [REM-25203] Apply selector-preserving method swizzling to UIControl/UICollectionViewCell/UITableViewCell. [REM-25202] [iOS][Perf] Apply selector-preserving method swizzling to UIViewController. [REM-25200] Apply selector-preserving method swizzling to UIWebView (rakutentech#21) REM-25339 Add RPTForceTrackingEnabled usage description to README (rakutentech#20) REM-25345 Add tests for implementationWithBlock swizzling (rakutentech#19) REM-24817 Fix swizzle crash when there is no original method and the swizzle gets added twice (rakutentech#16) REM-25242 Fix redirect handling in custom protocol (rakutentech#15) REM-25138 Debug builds should always send performance data if config API response is valid (rakutentech#12) feat(swizzling): REM-24817 Rewrite WKWebView swizzling (rakutentech#14) REM-25817 Fix crash that occurs when info.plist keys are missing by failing safely in release builds (rakutentech#13) REM-25128 Prepare for publishing to CocoaPods (rakutentech#10)
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Verified in iOS 11 simulator by confirming that PointClub bugs are fixed.