-
-
Notifications
You must be signed in to change notification settings - Fork 399
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: set a 1MB event limit #2062
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/athens-research/athens/ktr2q4jeSewHwJtXihJVUbMzN2qB |
2f26baa
to
a1b4e8e
Compare
a1b4e8e
to
81198f4
Compare
Realized this later on: this limit also applies to db dump will cause our team server to stop working. We can just exclude db dump from the limit, but the question of what limit there is for db dump still remains. |
I will need to test we don't have any 1mb event before merging by replaying the team log. |
When testing, I can't even paste 80KB of text into the app. |
LGTM |
No description provided.