Skip to content

New serverless pattern - apigw-lambda-transcribe #2713

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

archiev4
Copy link
Contributor

@archiev4 archiev4 commented Apr 16, 2025

Issue #2714, if available:

Description of changes:

A new Terraform pattern which does audio-to-text transcriptions using Lambda. This uses API Gateway-Lambda to generate an S3 pre-signed URL and upon uploading an object using that URL, another Lambda function will be invoked to start the Transcribe transcription job. The result is uploaded to the output S3 bucket.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Archana Venkitaramanan added 2 commits April 17, 2025 18:04
Copy link
Contributor

@marcojahn marcojahn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @archiev4 thank you for your contribution. Can you please review the PR comments and fix? - TY

@archiev4
Copy link
Contributor Author

archiev4 commented Apr 22, 2025

Hi @marcojahn

Thank you for looking into this. I have made the changes in the latest commit. I have also changed audio to video files so that it works for subtitle generation as well.

Please do take a look and let me know!

Copy link
Contributor

@marcojahn marcojahn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @archiev4, TY for the updates. 2 minor changes requested. The missing whitespaces seem to break the readme/markdown renderer. Can you pls double check. After that all good to go.

@archiev4
Copy link
Contributor Author

Hi @marcojahn, apologies for missing that. I have put the Curl command in one single line. Hope it is fine now

Copy link
Contributor

@marcojahn marcojahn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @archiev4 for your contribution. The PR will be merged by a Serverless DA soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants