Skip to content
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

[CAL-3082] eventTypes/getByViewer regression on v3.7.12 #13549

Closed
zomars opened this issue Feb 6, 2024 · 2 comments · Fixed by #13552
Closed

[CAL-3082] eventTypes/getByViewer regression on v3.7.12 #13549

zomars opened this issue Feb 6, 2024 · 2 comments · Fixed by #13552
Assignees
Labels
api area: API, enterprise API, access token, OAuth 🐛 bug Something isn't working performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive 📉 regressing This used to work. Now it doesn't anymore. Urgent Created by Linear-GitHub Sync
Milestone

Comments

@zomars
Copy link
Member

zomars commented Feb 6, 2024

rel #13002

It seems like we're having a increased payload on latest release. It is causing this endpoint to fail. This is what we have on the runtime logs:

[PERF]: getByViewer(109147) took 402.34358200000133ms
API response for /api/trpc/eventTypes/getByViewer?batch=1&input={"0":{"json":{"filters":{"teamIds":[REDACTED],"upIds":null,"userIds":null}},"meta":{"values":{"filters.upIds":["undefined"],"filters.userIds":["undefined"]}}}} exceeds 4MB. API Routes are meant to respond quickly. https://nextjs.org/docs/messages/api-routes-response-size-limit
[ERROR] [1707184498477] LAMBDA_RUNTIME Failed to post handler success response. Http response code: 413.

There's also a potential leakage of password of your team members as well.

From SyncLinear.com | CAL-3082

@zomars zomars added the Urgent Created by Linear-GitHub Sync label Feb 6, 2024
@maige-app maige-app bot added api area: API, enterprise API, access token, OAuth performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive labels Feb 6, 2024
@zomars zomars added 🐛 bug Something isn't working 📉 regressing This used to work. Now it doesn't anymore. labels Feb 6, 2024
@zomars
Copy link
Member Author

zomars commented Feb 6, 2024

Testing locally seems like the payload is now 34MB 🤯

image

@zomars
Copy link
Member Author

zomars commented Feb 6, 2024

Reduced to 1MB in #13552

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api area: API, enterprise API, access token, OAuth 🐛 bug Something isn't working performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive 📉 regressing This used to work. Now it doesn't anymore. Urgent Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants