Skip to content
This repository was archived by the owner on Oct 23, 2023. It is now read-only.

Conversation

@s0undt3ch
Copy link
Contributor

Refs #339.

@s0undt3ch
Copy link
Contributor Author

I was able to trigger the timeout once using my cloned fork of raven.
This is a small workaround, though, I've been unable to repeat my ssl read errors.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0%) when pulling 698dcf4 on s0undt3ch:master into d201373 on getsentry:master.

@dcramer
Copy link
Member

dcramer commented Aug 6, 2013

I dont recall if we prevent you from submitting the same event twice (based on an event id), but if not this would cause you to submit it repeatedly in real-world situations.

This also would cause people to DDOS getsentry.com as soon as they hit a 429 (rate limited error), which is likely an HTTPError.

Realistically I dont think this is a good solution to "my webserver timed out". It's just log data, and there will be more of it. If the timeout is the problem we should address that, or if you're hosting your own Sentry you should enable the cache + queue.

@dcramer dcramer closed this Aug 6, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants