Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: options request #311

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

feat: options request #311

wants to merge 18 commits into from

Conversation

YoloMao
Copy link
Collaborator

@YoloMao YoloMao commented Apr 25, 2024

  • 添加事件上报前的预检请求(会话级)
  • 添加是否开启预检配置项 requestPreflight,默认值为 false (仅作用于 CDP)
  • 添加备用域名配置项 alternateDataCollectionServerHost,预检失败 (HTTPCode 403) 降级至该备用域名
  • 每日流量上报限制默认值从 10MB 提高到 20MB
  • 事件上报返回 413 则删除整个请求包对应的事件
  • 事件上报返回 403 则再次启动预检
  • 打开采集开关将根据预检状态是否为待预检来启动预检;关闭采集开关时,不会进行预检,并设置当前预检状态为待预检

Copy link

codecov bot commented Apr 25, 2024

Codecov Report

Attention: Patch coverage is 87.77778% with 22 lines in your changes are missing coverage. Please review.

Project coverage is 83.77%. Comparing base (76da54a) to head (eaa26ec).

Files Patch % Lines
Modules/Preflight/GrowingNetworkPreflight.m 78.65% 19 Missing ⚠️
GrowingTrackerCore/Event/GrowingEventManager.m 95.23% 1 Missing ⚠️
...les/Advertising/AppleSearchAds/GrowingAsaFetcher.m 66.66% 1 Missing ⚠️
Modules/Preflight/Request/GrowingPFRequest.m 95.23% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #311      +/-   ##
==========================================
+ Coverage   83.07%   83.77%   +0.70%     
==========================================
  Files         137      141       +4     
  Lines        8057     8215     +158     
==========================================
+ Hits         6693     6882     +189     
+ Misses       1364     1333      -31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@YoloMao YoloMao force-pushed the feat/options_request branch 3 times, most recently from 6f64568 to 92e9038 Compare May 7, 2024 08:08
Copy link

sonarcloud bot commented May 31, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
3.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants