Skip to content

Comments

chore: Merge hotfix changes from v8.57.0 into main#6419

Merged
philprime merged 7 commits intomainfrom
philprime/v8.x-cleaned
Oct 15, 2025
Merged

chore: Merge hotfix changes from v8.57.0 into main#6419
philprime merged 7 commits intomainfrom
philprime/v8.x-cleaned

Conversation

@philprime
Copy link
Member

@philprime philprime commented Oct 14, 2025

Merges changes released on the hotfix branch to main.

#skip-changelog

Closes #6420

@philprime philprime self-assigned this Oct 14, 2025
@philprime philprime force-pushed the philprime/v8.x-cleaned branch from 642ddf8 to b3952ce Compare October 14, 2025 10:13
@philprime philprime changed the title chore: Merge hotfix changes from v8.x into main chore: Merge hotfix changes from v8.57.0 into main Oct 14, 2025
@philprime philprime marked this pull request as ready for review October 14, 2025 11:45
Copy link
Member

@philipphofmann philipphofmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM when CI is green.

@philprime philprime enabled auto-merge (squash) October 14, 2025 14:07
@codecov
Copy link

codecov bot commented Oct 14, 2025

Codecov Report

❌ Patch coverage is 97.76786% with 5 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@c1254d8). Learn more about missing BASE report.
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...Replay/SentrySessionReplayEnvironmentChecker.swift 95.698% 4 Missing ⚠️
...wift/Helper/InfoPlist/SentryInfoPlistWrapper.swift 97.560% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             main     #6419   +/-   ##
========================================
  Coverage        ?   86.895%           
========================================
  Files           ?       445           
  Lines           ?     37505           
  Branches        ?     17447           
========================================
  Hits            ?     32590           
  Misses          ?      4870           
  Partials        ?        45           
Files with missing lines Coverage Δ
SentryTestUtils/TestInfoPlistWrapper.swift 100.000% <100.000%> (ø)
...estUtils/TestSessionReplayEnvironmentChecker.swift 100.000% <100.000%> (ø)
Sources/Sentry/SentryDependencyContainer.m 90.163% <100.000%> (ø)
Sources/Sentry/SentryMeta.m 50.000% <ø> (ø)
Sources/Sentry/SentrySessionReplayIntegration.m 87.364% <100.000%> (ø)
...tegrations/SessionReplay/SentrySessionReplay.swift 90.965% <100.000%> (ø)
Sources/Swift/SentryExperimentalOptions.swift 84.615% <100.000%> (ø)
...wift/Helper/InfoPlist/SentryInfoPlistWrapper.swift 97.560% <97.560%> (ø)
...Replay/SentrySessionReplayEnvironmentChecker.swift 95.698% <95.698%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c1254d8...a1b19dd. Read the comment docs.

@philprime philprime requested a review from Copilot October 14, 2025 18:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR merges hotfix changes from v8.57.0 into main to address critical PII leaks in Session Replay on iOS 26 with Liquid Glass rendering. The changes add defensive programming to automatically disable Session Replay in unreliable environments unless explicitly overridden.

  • Adds environment checking to detect iOS 26+ with Xcode 26+ builds that may cause masking failures
  • Introduces experimental option to override the automatic disabling in unreliable environments
  • Updates version numbers across all configuration files and package manifests

Reviewed Changes

Copilot reviewed 40 out of 40 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
sdk_api.json Adds new experimental option API definition
Sources/Swift/SentryExperimentalOptions.swift Defines enableSessionReplayInUnreliableEnvironment flag
Sources/Swift/Integrations/SessionReplay/SentrySessionReplay.swift Implements environment checking and conditional disabling
Sources/Swift/Integrations/SessionReplay/SentrySessionReplayEnvironmentChecker.swift Core logic for detecting unreliable environments
Sources/Swift/Helper/InfoPlist/*.swift New utilities for reading Info.plist values
Tests/**/*.swift Comprehensive test coverage for new functionality
Configuration files Version bumps to 8.57.0

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1218.45 ms 1245.72 ms 27.28 ms
Size 23.75 KiB 1004.68 KiB 980.94 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
e3767a1 1224.20 ms 1257.16 ms 32.96 ms
b115f82 1212.96 ms 1251.92 ms 38.96 ms
438caec 1230.02 ms 1250.00 ms 19.98 ms
7d23639 1237.93 ms 1243.04 ms 5.11 ms
e64d3d4 1241.90 ms 1260.10 ms 18.20 ms
5200f5b 1231.73 ms 1254.35 ms 22.62 ms
701acf0 1230.24 ms 1255.60 ms 25.36 ms
2675d3c 1218.94 ms 1239.36 ms 20.42 ms
80538ca 1216.70 ms 1253.92 ms 37.22 ms
884b224 1233.41 ms 1259.50 ms 26.09 ms

App size

Revision Plain With Sentry Diff
e3767a1 23.75 KiB 913.14 KiB 889.39 KiB
b115f82 23.75 KiB 989.04 KiB 965.30 KiB
438caec 23.74 KiB 913.14 KiB 889.40 KiB
7d23639 23.75 KiB 891.01 KiB 867.26 KiB
e64d3d4 23.75 KiB 855.37 KiB 831.62 KiB
5200f5b 23.75 KiB 933.33 KiB 909.58 KiB
701acf0 23.75 KiB 987.96 KiB 964.21 KiB
2675d3c 23.75 KiB 928.16 KiB 904.41 KiB
80538ca 23.75 KiB 989.99 KiB 966.24 KiB
884b224 23.75 KiB 879.60 KiB 855.86 KiB

@philprime philprime merged commit caa7e70 into main Oct 15, 2025
195 of 199 checks passed
@philprime philprime deleted the philprime/v8.x-cleaned branch October 15, 2025 08:39
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.

chore: Merge hotfix changes from v8.57.0 into main

3 participants