extension handling will let in no-extension paths

aws-samples / amazon-bedrock-audio-summarizer

An automated way to transcribe and summarize media files using Amazon S3, AWS Lambda, Amazon Transcribe, and Amazon Bedrock.

MIT No Attribution

96 stars 20 forks source link

Closed burhan closed 3 weeks ago

burhan commented 1 month ago

The default logic for extracting the extension, will fail if the path has no extension:

>>> "source/foo".split(".")[-1]
'source/foo'

using os.path.splitext provides better extension handling, as it will catch the above case.

erikhopf commented 3 weeks ago

@burhan going to close this issue since it looks like @jbnunn merged your PR. Happy to re-open this if needed, just let me know.