Madison Swain-Bowden

Results 209 comments of Madison Swain-Bowden

That's a good point about potentially overwriting the tags rather than merging them! I wonder though, if it might actually be better to store all of the tags but (similar...

These are all really good points, and I think asking the question of _what do we hope to get from this data_ is always a good place to start before...

I have seen this and will respond to this discussion when I have time!

Thanks for this awesome analysis! > I think the thing to do is lower the timeout to 4 seconds, which is really conservative, but much better than what it is...

@svenevs @Lemmons I'm seeing this as well - I can take a stab at it to try and find a solution, but I wanted to check in. Are you two...

We're seeing that this now raises a `SuspiciousOperation` error within the Oauth backend per this Sentry issue: https://openverse.sentry.io/share/issue/eb602c45bcb34dc8947e660d793eea2e/

Reopening as this occurred again recently (example URL: http://api.openverse.org/v1/images/?q=73%25%20of%20Arkansans%20think%20that%20crime%20is%20on%20the%20rise%20in%20their%20state, which unencoded reads as `73% of Arkansans think that crime is on the rise in their state`). Sentry link: https://openverse.sentry.io/share/issue/6031f339cffe4045b0e6198e251bf2d3/

Some related issues I've found: - https://github.com/jazzband/django-oauth-toolkit/issues/954 - https://github.com/oauthlib/oauthlib/issues/662 It seems that the former resulted in https://github.com/jazzband/django-oauth-toolkit/pull/963 which fixed cases of this for `oauth2_provider/backends.py`: https://github.com/jazzband/django-oauth-toolkit/blob/102c85141ec44549e17080c676292e79e5eb46cc/oauth2_provider/backends.py#L18-L27 But not for the `rest_framework`...

Hoping this can be addressed by: https://github.com/jazzband/django-oauth-toolkit/pull/1444