-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add business logic to handler #7
Conversation
@@ -0,0 +1,247 @@ | |||
import axios from 'axios'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I initially used node's Undici fetch
for all fetch calls, but kept hitting a timeout issue (perhaps related?). The timeout might just have been due to something with my connection though, so after we've tested it in other environments we might want to revert to using fetch
(so as to have fewer dependencies and greater compatibility with other runtimes, e.g. Deno)
da00532
to
1d954ea
Compare
What does this change?
An initial pass at implementing the business logic for the pressreader job.
It then returns a list of sections and article ids, to be saved to S3 by the handler function.
How to test
resources.sh
(but make sure not to commit this!)npm run dev
.config.ts
.How can we measure success?
This is part of migrating the existing service from on-prem to AWS Lambda.