[Android] [SentryHttpClientException] Exclusions

Integration

sentry-android-okhttp

Build System

Gradle

AGP Version

8.2.2

Proguard

Enabled

Version

7.4.0

Steps to Reproduce

I'm getting a number of 504 errors from https://res.cloudinary.com that I want to suppress. I am trying to suppress the error results from this domain when I add the Sentry interceptor. E.g.

  OkHttpClient.Builder().addInterceptor(SentryOkHttpInterceptor(failedRequestTargets = 
    listOf("^(?!(.*.cloudinary.com))"))).build()

Am I defining the regex incorrectly? I can't find any decent documentation on this.

Expected Result

No errors are returned for given domain name.

Actual Result

Sentry console filled with SentryHttpClientException: HTTP Client Error with status code: 504 for this domain.

@barry-irvine

Hi @barry-irvine, thanks for reaching out!

~~failedRequestTargets can either be a plain string or an regex. But there's no need to define regex groups aka (). So in your case this should work fine:~~

My bad, I misunderstood your question. The regex supplied in failedRequestTargets needs to match a full URL, including the scheme and potential url parameters. Something like this should work:

OkHttpClient.Builder().addInterceptor(
    SentryOkHttpInterceptor(failedRequestTargets = listOf("^((http|https)://)?(?!.*\.?cloudinary\.com).*$"))
).build()

@barry-irvine

@barry-irvine did you manage to verify whether the updated answer works for your case?

@courville

@courville thanks for chiming in here, do you have failedRequestTargets configured in your app?

This behavior is likely due to changes introduced in 7.1.0(see changelog), where the SDK would start reporting errors, which it previously didn't.

thanks for the feedback and nova-video-player/aos-Video@930b65b did the trick.

This was quite a change of behavior and unexpected.

Sorry for the noise.

@courville

@courville Apologies for the confusion - 7.1.0 fixed an issue where before that versoin, errors wouldn't be reported even though we communicated that they would. In general we don't change behavior like this outside of major version bumps.

Closing this for now, if you think there's still an issue simply re-open this ticket.

@markushi

@markushi I've finally got the updated code in production and can still see the 504 cloudinary errors in Sentry.

This was the final implementation. Did I do something wrong?

@Provides
@CommonOkHttp
fun provideCommonOkHttpClient(): OkHttpClient =
    OkHttpClient.Builder()
        .addInterceptor(
            SentryOkHttpInterceptor(failedRequestTargets = listOf("^((http|https)://)?(?!.*\\.?cloudinary\\.com).*$"))
        ).build()

And yet I see a 504 error for "https://res.cloudinary.com/dtljonz0f/image/upload/c_fill,g_auto,h_333,w_250/gc-v1/las-vegas/Favorite-Bistro7206_0.jpg" in the console. It looks like we get the 504s when the user's device is offline and Coil (the library we use for image processing) doesn't even bother to send the image request to Cloudinary because it knows they're offline so it just returns a 504 straightaway.

@markushi I've finally got the updated code in production and can still see the 504 cloudinary errors in Sentry.

This was the final implementation. Did I do something wrong?
@Provides
@CommonOkHttp
fun provideCommonOkHttpClient(): OkHttpClient =
    OkHttpClient.Builder()
        .addInterceptor(
            SentryOkHttpInterceptor(failedRequestTargets = listOf("^((http|https)://)?(?!.*\\.?cloudinary\\.com).*$"))
        ).build()
And yet I see a 504 error for "https://res.cloudinary.com/dtljonz0f/image/upload/c_fill,g_auto,h_333,w_250/gc-v1/las-vegas/Favorite-Bistro7206_0.jpg" in the console. It looks like we get the 504s when the user's device is offline and Coil (the library we use for image processing) doesn't even bother to send the image request to Cloudinary because it knows they're offline so it just returns a 504 straightaway.

For anyone who was having this issue of non-existent 504s being reported to Sentry, it was just fixed in 8.40.0 here: #5299

Navigation Menu

Repository navigation

[Android] [SentryHttpClientException] Exclusions #3244

Integration

Build System

AGP Version

Proguard

Version

Steps to Reproduce

Expected Result

Actual Result

15 remaining items

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

[Android] [SentryHttpClientException] Exclusions #3244

Description

Integration

Build System

AGP Version

Proguard

Version

Steps to Reproduce

Expected Result

Actual Result

Activity

markushi commented on Mar 6, 2024

15 remaining items

romtsn commented on Mar 27, 2024

courville commented on Mar 27, 2024

kahest commented on Mar 28, 2024

markushi commented on Apr 3, 2024

barry-irvine commented on Apr 15, 2024

angusholder commented on Apr 23, 2026

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions