Feat: Add redirected_status_code to CrawlResult #1435
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Add redirected_status_code to CrawlResult so that there is a way to tell whether the redirected response is successful or not
Fixes #1434
List of files changed and why
_crawl_web
async_response
to thecrawl_result
redirected_status_code
field toCrawlResult
andAsyncCrawlResponse
How Has This Been Tested?
I have tested with an example site containing working and failing redirect links, and the redirected status code is correctly set
Checklist:
Results
Here is the new response from the fail and success redirect site (some fields removed for conciseness)