Skip to content

Conversation

@alexander-alderman-webb
Copy link
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Oct 21, 2025

Description

See #4902 (comment).

We successfully trialed the option as part of version 2.42.0 of the SDK in Sentry: getsentry/sentry#101502

Issues

Reminders

@codecov
Copy link

codecov bot commented Oct 21, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
23712 1 23711 1713
View the top 1 failed test(s) by shortest run time
tests.integrations.redis.test_redis_cache_module::test_key_as_string[key7-]
Stack Traces | 0.006s run time
.../integrations/redis/test_redis_cache_module.py:318: in test_key_as_string
    assert _key_as_string(key) == expected_key
E   AssertionError: assert 'Ǟ\x04$k\x06Hԗ\x01ݒ{/\x1f\x14' == ''
E     + Ǟ#x04$k#x06Hԗ#x01ݒ{/#x1F#x14

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@alexander-alderman-webb alexander-alderman-webb marked this pull request as draft October 21, 2025 06:46
@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review October 21, 2025 07:08
@alexander-alderman-webb alexander-alderman-webb merged commit dcd94ed into master Oct 21, 2025
119 of 121 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/enable-http-request-source-by-default branch October 21, 2025 08:44
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.

3 participants