Merge pull request #264 from wangliangliang2/master

fix delegate sometimes not release

Author: longbai

QiniuSDK/Http/QNSessionManager.m

@@ -58,13 +58,6 @@ - (void)URLSession:(NSURLSession *)session task:(NSURLSessionTask *)task
     if (_cancelBlock && _cancelBlock()) {
         [_task cancel];
     }
-
-    float progress = (float)totalBytesSent / totalBytesExpectedToSend;
-    if (progress == 1.0f) {
-        self.task = nil;
-        self.cancelBlock = nil;
-        self.progressBlock = nil;
-    }
 }
 
 @end
@@ -203,7 +196,7 @@ - (void)sendRequest2:(NSMutableURLRequest *)request
     QNInternalProgressBlock progressBlock2 = ^(long long totalBytesWritten, long long totalBytesExpectedToWrite) {
         progressBlock(totalBytesWritten, totalBytesExpectedToWrite);
     };
-    QNProgessDelegate *delegate = (QNProgessDelegate *)_httpManager.delegate;
+    __block QNProgessDelegate *delegate = (QNProgessDelegate *)_httpManager.delegate;
     delegate.progressBlock = progressBlock2;
     NSURLSessionUploadTask *uploadTask = [_httpManager uploadTaskWithRequest:request fromData:nil completionHandler:^(NSData *_Nullable data, NSURLResponse *_Nullable response, NSError *_Nullable error) {
 
@@ -224,6 +217,9 @@ - (void)sendRequest2:(NSMutableURLRequest *)request
         } else {
             info = [QNSessionManager buildResponseInfo:httpResponse withError:error withDuration:duration withResponse:data withHost:domain withIp:ip];
         }
+        delegate.task = nil;
+        delegate.cancelBlock = nil;
+        delegate.progressBlock = nil;
         completeBlock(info, resp);
     }];
     delegate.task = uploadTask;

QiniuSDKTests/QNFileRecorderTest.m

@@ -82,7 +82,7 @@ - (void) template:(int)size pos:(float)pos {
         if (percent < pos - 256.0 / size) {
             failed = YES;
         }
-        NSLog(@"continue progress %f", percent);
+        NSLog(@"continue progress %f,%f", percent,pos - 256.0 / size);
     }
                                         params:nil
                                       checkCrc:NO
@@ -92,6 +92,7 @@ - (void) template:(int)size pos:(float)pos {
         info = i;
     }
                  option:opt];
+    NSLog(@"failed: %@",failed ? @"YES" : @"NO");
     AGWW_WAIT_WHILE(key == nil, 60 * 30);
     NSLog(@"info %@", info);
     XCTAssert(info.isOK, @"Pass");
@@ -132,7 +133,7 @@ - (void)test8M {
     if (_inTravis) {
         return;
     }
-    [self template:8 * 1024 + 1 pos:0.8];
+    [self template:8 * 1024 + 1 pos:0.7];
 }
 
 //#endif

README.md

@@ -13,14 +13,14 @@
 通过 CocoaPods
 
 ```ruby
-pod "Qiniu", "~> 7.2"
+pod "Qiniu", "~> 7.2" 
 ```
 
 ## 运行环境
 
 |               Qiniu SDK 版本               | 最低 iOS版本 | 最低 OS X 版本 |     Notes     |
 | :--------------------------------------: | :------: | :--------: | :-----------: |
-|         7.2.x          |  iOS 7   | OS X 10.9  | Xcode 最低版本 6. |
+|                  7.2.x                   |  iOS 7   | OS X 10.9  | Xcode 最低版本 6. |
 |         7.1.x / AFNetworking-3.x         |  iOS 7   | OS X 10.9  | Xcode 最低版本 6. |
 | [7.0.x / AFNetworking-2.x](https://github.com/qiniu/objc-sdk/tree/7.0.x/AFNetworking-2.x) |  iOS 6   | OS X 10.8  | Xcode 最低版本 5. |
 | [7.x / AFNetworking-1.x](https://github.com/qiniu/objc-sdk/tree/AFNetworking-1.x) |  iOS 5   | OS X 10.7  | Xcode 最低版本 5. |