Update OpenAPI schema for /crawl/active endpoint created_at field #47
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.
Add created_at field to /crawl/active endpoint response
Summary
Adds a new
created_at
field to the/crawl/active
API endpoint response, showing when each crawl was created. The field returns an ISO 8601 timestamp string converted from the existing Unix timestamp stored in Redis.Key Changes:
OngoingCrawlsResponse
TypeScript type to includecreated_at: string
ongoingCrawlsController
to returncreated_at
field by converting Unix timestamp to ISO stringReview & Testing Checklist for Human
created_at
field - Test the live/crawl/active
endpoint to confirm the field appears in responsescreatedAt
is undefined/null on stored crawl objects (currently falls back toDate.now()
)Recommended test plan:
/crawl/active
to see the crawl withcreated_at
fieldDiagram
Notes
createdAt
is undefined, it falls back toDate.now()
- this may not be the desired behavior for all use cases