💥 [RUM-6816] remove sendLogsAfterSessionExpiration

[thomas-lebeau]

Motivation

Remove sendLogsAfterSessionExpiration option and makes sending logs after session expiration the default behavior

Changes

Testing

• Local
• Staging
• Unit
• End to end

---

I have gone over the contributing documentation.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.58%. Comparing base (89107f0) to head (01aa85d).
Report is 2 commits behind head on v6.

Additional details and impacted files

@@            Coverage Diff             @@
##               v6    #3183      +/-   ##
==========================================
+ Coverage   93.56%   93.58%   +0.01%     
==========================================
  Files         279      279              
  Lines        7478     7477       -1     
  Branches     1689     1687       -2     
==========================================
  Hits         6997     6997              
+ Misses        481      480       -1     

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

[cit-pr-commenter]

Bundles Sizes Evolution

📦 Bundle Name	Base Size	Local Size	𝚫	𝚫%	Status
Rum	151.96 KiB	151.96 KiB	0 B	0.00%	✅
Logs	50.76 KiB	50.60 KiB	-164 B	-0.32%	✅
Rum Slim	102.16 KiB	102.16 KiB	0 B	0.00%	✅
Worker	25.16 KiB	25.16 KiB	0 B	0.00%	✅

🚀 CPU Performance

Action Name	Base Average Cpu Time (ms)	Local Average Cpu Time (ms)	𝚫
addglobalcontext	0.002	0.002	0.000
addaction	0.042	0.040	-0.001
addtiming	0.001	0.001	-0.000
adderror	0.037	0.042	0.005
startstopsessionreplayrecording	0.895	0.870	-0.025
startview	1.337	1.196	-0.142
logmessage	0.020	0.022	0.001

🧠 Memory Performance

Action Name	Base Consumption Memory (bytes)	Local Consumption Memory (bytes)	𝚫 (bytes)
addglobalcontext	8.88 KiB	9.40 KiB	529 B
addaction	41.79 KiB	39.49 KiB	-2356 B
addtiming	8.35 KiB	7.46 KiB	-915 B
adderror	44.47 KiB	45.61 KiB	1.14 KiB
startstopsessionreplayrecording	5.29 KiB	7.41 KiB	2.12 KiB
startview	406.69 KiB	404.67 KiB	-2067 B
logmessage	41.53 KiB	41.72 KiB	202 B

🔗 RealWorld

[thomas-lebeau]

This was not correct. According to the implementation, if a session is not tracked, it should return undefined regardless of returnInactive option