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.

Sign up for GitHub

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

Jump to bottom

improve inputdata checks #6558

Open
mahf708 opened this issue Aug 21, 2024 · 3 comments
Open

improve inputdata checks #6558

mahf708 opened this issue Aug 21, 2024 · 3 comments
Labels
help wanted Testing Anything related to unit/system tests

Comments

@mahf708
Copy link
Contributor

mahf708 commented Aug 21, 2024

Per @rljacob: We got this advice “If you don’t need to download it, you can send a HTTP HEAD request, like these examples https://gist.github.com/dehowell/884204 or this https://stackoverflow.com/a/35881430
That would be easier on the webserver and should run faster.”
@mahf708 mahf708 added the Testing Anything related to unit/system tests label Aug 21, 2024
@mahf708 mahf708 self-assigned this Aug 21, 2024
@bartgol
Copy link
Contributor

bartgol commented Aug 21, 2024

Out of curiosity: does this address the 403 Forbidden error as well?

@rljacob
Copy link
Member

rljacob commented Aug 21, 2024

No. Those were from blocks put in at the web server. This is just good practice going forward.

@mahf708
Copy link
Contributor Author

mahf708 commented Aug 21, 2024

Yeah, just to lessen the pressure on the server

@mahf708 mahf708 removed their assignment Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Testing Anything related to unit/system tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rljacob @bartgol @mahf708