Merge pull request #938 from jcbjoe/jg/optional-authentication-polly

Changed Polly authentication params to be optional
This commit is contained in:
Mark Backman
2025-01-07 15:37:23 -05:00
committed by GitHub
2 changed files with 7 additions and 3 deletions

View File

@@ -16,6 +16,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Added `enable_prejoin_ui`, `max_participants` and `start_video_off` params - Added `enable_prejoin_ui`, `max_participants` and `start_video_off` params
to `DailyRoomProperties`. to `DailyRoomProperties`.
### Changed
- api_key, aws_access_key_id and region are no longer required parameters for the PollyTTSService (AWSTTSService)
### Fixed ### Fixed
- Fixed an issue where `OpenAIRealtimeBetaLLMService` audio chunks were hitting - Fixed an issue where `OpenAIRealtimeBetaLLMService` audio chunks were hitting

View File

@@ -119,9 +119,9 @@ class PollyTTSService(TTSService):
def __init__( def __init__(
self, self,
*, *,
api_key: str, api_key: Optional[str] = None,
aws_access_key_id: str, aws_access_key_id: Optional[str] = None,
region: str, region: Optional[str] = None,
voice_id: str = "Joanna", voice_id: str = "Joanna",
sample_rate: int = 24000, sample_rate: int = 24000,
params: InputParams = InputParams(), params: InputParams = InputParams(),