Skip to content

Commit 4817308

Browse files
authored
Merge pull request #419 from YangSen-qn/develop
修复log report统计问题
2 parents 2e04bbf + 9081f8f commit 4817308

File tree

8 files changed

+20
-12
lines changed

8 files changed

+20
-12
lines changed

CHANGELOG.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
#Changelog
2+
## 8.0.1 (2020-09-08)
3+
* 修复日志统计问题
24

35
## 8.0.0 (2020-08-06)
46
## 重构
@@ -10,14 +12,18 @@
1012

1113
8.0.0版本是在7.x版本基础上,在保证上传接口不变的条件下进行的升级,代码修改信息如下:
1214
- 增加的内部使用类:GlobalConfiguration(dns配置类)、HttpRegionRequest、UploadRequestState等
13-
- 废弃类:UploadInfoReporter
15+
- 废弃类:HttpResponseInfo、UploadInfoReporter
1416
- 修改了一些7.x内部使用类的API
1517
- UploadManager上传接口进行了少量调整
1618

1719
如果您并未对7.x SDK 进行过深度定制,而是直接对接QNUploadManager进行上传,那么SDK版本升级只需要花费几分钟的时间进行即可。
1820

1921
详细情况请参考:[Android_Changes_from_7.x_to_8.0.0.md](./doc/Android_Changes_from_7.x_to_8.0.0.md)
2022

23+
# 7.7.0
24+
* 压缩上报的日志内容
25+
* 获取AutoZone信息接口升级至v4
26+
2127
# 7.6.7
2228
* 修复创建dns缓存文件失败问题
2329

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ https://github.com/qiniudemo/qiniu-lab-android
2727
| 7.0.7 | Android 2.2+ | android-async-http 1.4.8 |
2828

2929
### 注意
30-
* 推荐使用最新版:8.0.0
30+
* 推荐使用最新版:8.0.1
3131
* AndroidNetwork.getMobileDbm()可以获取手机信号强度,需要如下权限(API>=18时生效)
3232
```
3333
<uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION"/>

library/library.iml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,16 +31,16 @@
3131
<output-test url="file://$MODULE_DIR$/build/intermediates/javac/debugUnitTest/classes" />
3232
<exclude-output />
3333
<content url="file://$MODULE_DIR$">
34+
<sourceFolder url="file://$MODULE_DIR$/build/generated/renderscript_source_output_dir/debug/compileDebugRenderscript/out" isTestSource="false" generated="true" />
35+
<sourceFolder url="file://$MODULE_DIR$/build/generated/source/buildConfig/debug" isTestSource="false" generated="true" />
3436
<sourceFolder url="file://$MODULE_DIR$/build/generated/ap_generated_sources/debug/out" isTestSource="false" generated="true" />
3537
<sourceFolder url="file://$MODULE_DIR$/build/generated/aidl_source_output_dir/debug/compileDebugAidl/out" isTestSource="false" generated="true" />
36-
<sourceFolder url="file://$MODULE_DIR$/build/generated/source/buildConfig/debug" isTestSource="false" generated="true" />
37-
<sourceFolder url="file://$MODULE_DIR$/build/generated/renderscript_source_output_dir/debug/compileDebugRenderscript/out" isTestSource="false" generated="true" />
3838
<sourceFolder url="file://$MODULE_DIR$/build/generated/res/rs/debug" type="java-resource" generated="true" />
3939
<sourceFolder url="file://$MODULE_DIR$/build/generated/res/resValues/debug" type="java-resource" generated="true" />
4040
<sourceFolder url="file://$MODULE_DIR$/build/generated/ap_generated_sources/debugAndroidTest/out" isTestSource="true" generated="true" />
4141
<sourceFolder url="file://$MODULE_DIR$/build/generated/aidl_source_output_dir/debugAndroidTest/compileDebugAndroidTestAidl/out" isTestSource="true" generated="true" />
42-
<sourceFolder url="file://$MODULE_DIR$/build/generated/source/buildConfig/androidTest/debug" isTestSource="true" generated="true" />
4342
<sourceFolder url="file://$MODULE_DIR$/build/generated/renderscript_source_output_dir/debugAndroidTest/compileDebugAndroidTestRenderscript/out" isTestSource="true" generated="true" />
43+
<sourceFolder url="file://$MODULE_DIR$/build/generated/source/buildConfig/androidTest/debug" isTestSource="true" generated="true" />
4444
<sourceFolder url="file://$MODULE_DIR$/build/generated/res/rs/androidTest/debug" type="java-test-resource" generated="true" />
4545
<sourceFolder url="file://$MODULE_DIR$/build/generated/res/resValues/androidTest/debug" type="java-test-resource" generated="true" />
4646
<sourceFolder url="file://$MODULE_DIR$/build/generated/ap_generated_sources/debugUnitTest/out" isTestSource="true" generated="true" />

library/src/main/java/com/qiniu/android/collect/ReportItem.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,8 @@ public static String requestReportErrorType(ResponseInfo responseInfo){
123123
errorType = "network_error";
124124
} else if (responseInfo.statusCode == ResponseInfo.TimedOut){
125125
errorType = "timeout";
126+
} else if (responseInfo.statusCode == ResponseInfo.UnknownHost){
127+
errorType = "unknown_host";
126128
} else if (responseInfo.statusCode == ResponseInfo.CannotConnectToHost){
127129
errorType = "cannot_connect_to_host";
128130
} else if (responseInfo.statusCode == ResponseInfo.NetworkConnectionLost){

library/src/main/java/com/qiniu/android/collect/UploadInfoReporter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ private void saveReportJsonString(final String jsonString){
113113
return;
114114
}
115115
}
116-
116+
117117
if (recorderFile.length() > config.maxRecordFileSize){
118118
return;
119119
}

library/src/main/java/com/qiniu/android/common/Constants.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33

44
public final class Constants {
5-
public static final String VERSION = "8.0.0";
5+
public static final String VERSION = "8.0.1";
66

77
public static final String UTF_8 = "utf-8";
88
}

library/src/main/java/com/qiniu/android/http/request/RequestTransaction.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import org.json.JSONException;
2121
import org.json.JSONObject;
2222

23-
import java.util.ArrayList;
2423
import java.util.HashMap;
2524
import java.util.List;
2625
import java.util.Set;
@@ -34,7 +33,7 @@ public class RequestTransaction {
3433
private final String userAgent;
3534

3635
private UploadRequestInfo requestInfo;
37-
private UploadRequestState requstState;
36+
private UploadRequestState requestState;
3837
private HttpRegionRequest regionRequest;
3938

4039

@@ -85,12 +84,13 @@ private RequestTransaction(Configuration config,
8584
private void initData(IUploadRegion targetRegion,
8685
IUploadRegion currentRegion){
8786

88-
this.requstState = new UploadRequestState();
87+
this.requestState = new UploadRequestState();
8988
this.requestInfo = new UploadRequestInfo();
9089
this.requestInfo.targetRegionId = targetRegion.getZoneInfo().getRegionId();
9190
this.requestInfo.currentRegionId = currentRegion.getZoneInfo().getRegionId();
9291
this.requestInfo.bucket = token.bucket;
93-
this.regionRequest = new HttpRegionRequest(config, uploadOption, token, currentRegion, this.requestInfo, this.requstState);
92+
this.requestInfo.key = this.key;
93+
this.regionRequest = new HttpRegionRequest(config, uploadOption, token, currentRegion, this.requestInfo, this.requestState);
9494
}
9595

9696

library/src/main/java/com/qiniu/android/http/request/UploadRequestInfo.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,6 @@ class UploadRequestInfo {
1717
String currentRegionId;
1818

1919
boolean shouldReportRequestLog(){
20-
return requestType.equals(RequestTypeUpLog);
20+
return !requestType.equals(RequestTypeUpLog);
2121
}
2222
}

0 commit comments

Comments
 (0)