fix(automate): cap getFailureLogs log downloads at 5 MB by SavioBS629 · Pull Request #474 · browserstack/mcp-server · GitHub
Skip to content

fix(automate): cap getFailureLogs log downloads at 5 MB - #474

Merged
gaurav-singh-9227 merged 3 commits into
browserstack:mainfrom
SavioBS629:fix/failure-logs-size-cap-min
Oct 8, 2026
Merged

gaurav-singh-9227 merged 3 commits into
browserstack:mainfrom
SavioBS629:fix/failure-logs-size-cap-min

Conversation

@SavioBS629

Copy link
Copy Markdown
Collaborator

A single very large session log was downloaded with no size limit and copied several times in memory. Log requests now set maxContentLength (axios aborts the transfer past it); apiClient returns a 413 for that case when raise_error is false, and the tool reports the log as too large instead of failing. Network logs without log.entries no longer throw.

A single very large session log was downloaded with no size limit and copied several times in memory. Log requests now set maxContentLength (axios aborts the transfer past it); apiClient returns a 413 for that case when raise_error is false, and the tool reports the log as too large instead of failing. Network logs without log.entries no longer throw.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

SavioBS629 and others added 2 commits October 8, 2026 18:41
… only

Local servers keep downloading logs of any size; the 10 MB cap applies when REMOTE_MCP is set.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@SavioBS629 SavioBS629 changed the title fix(automate): cap getFailureLogs log downloads at 10 MB fix(automate): cap getFailureLogs log downloads at 5 MB Oct 8, 2026
@gaurav-singh-9227
gaurav-singh-9227 merged commit 9b53d22 into browserstack:main Oct 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants