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

Check for raw param before adding header Content-disposition. #2506

Merged
merged 1 commit into from
Nov 2, 2022

Conversation

Siddhantshukla814
Copy link
Contributor

This is specifically for node wrapper because server was throwing 403 forbidden error due to header missing with the request.

If &raw=1 is passed with the API request it will allow file downloading without checking for header Content-disposition

@Siddhantshukla814 Siddhantshukla814 added the Type: Bug An issue or pull request relating to a bug in WebPageTest label Nov 2, 2022
@tkadlec tkadlec merged commit cbb37b1 into master Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug An issue or pull request relating to a bug in WebPageTest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants