Merged
Conversation
addfdb6 to
90122aa
Compare
90122aa to
136866e
Compare
sirknightj
approved these changes
Oct 15, 2025
unicornss
approved these changes
Oct 15, 2025
YPandas
added a commit
that referenced
this pull request
Oct 28, 2025
* WebRTC ingestion with multi-viewer support (#115) * feat: enable ingest media for viewers with audio-only restriction - Remove master-only restriction for ingest media functionality - Add validation to prevent viewers from using ingest media with video - Implement storage session support for viewers (audio-only) - Update WebRTC activity to skip video track creation for storage viewers * Update WebRtcActivity with current version - Replace WebRtcActivity.java with current working version - remove the fallback solution of null clientID * adding verbose logs back and remove overrides that only call super * changing visibility of KEY_SEND_VIDEO variable * refactor: extract storage client methods and update master validation logic - Extract storage client configuration into separate methods in WebRtcActivity - Add helper methods for better code clarity - Remove mandatory video requirement for masters in non-storage scenarios - Update viewer validation message for multi-viewer participants * feat: Add H.264 codec validation for WebRTC storage sessions - Show AlertDialog when H.264 encoder missing for master with ingest media - Show AlertDialog when H.264 decoder missing for storage session viewers - Add codecValidationFailed flag to prevent WebRTC initialization after validation failure - Add null check for audioManager in onDestroy() to prevent crash on early exit * feat: Add codec logging and final parameter to WebRtcActivity - Restore codec logging in onCreate() to show all available encoders/decoders for debugging - Add final modifier --------- Co-authored-by: Yuqi Huang <yuuqih@amazon.com> * feat: Add error dialog for storage session join failures (#117) - Show AlertDialog when JoinStorageSession API fails - Close WebRTC activity when user dismisses error dialog Co-authored-by: Yuqi Huang <yuuqih@amazon.com> * endpoint and credential configuration through .env file (#116) * Add .env file support for AWS credentials with production warnings • Load AWS credentials from .env file via BuildConfig • Skip authentication flow when env credentials present • Disable logout functionality for env credential users • Add production warning message for env credential usage • Maintain fallback to Cognito authentication when no env file * Add custom endpoint configuration via .env file • Load CUSTOM_ENDPOINT from .env file via BuildConfig * format improvement * changing variable name(CUSTOM_ENDPOINT --> CONTROL_PLANE_URI) * Add CustomCredentialsProvider to use AWS credentials from .env file * refactor: centralize AWS credentials validation logic - Make hasEnvCredentials() public static in KinesisVideoWebRtcDemoApp - Remove duplicate hasEnvSetting() method from SimpleNavActivity - Replace inline credential validation in StartUpActivity --------- Co-authored-by: Yuqi Huang <yuuqih@amazon.com> * chore: upgrade AWS Android SDK from 2.75.0 to 2.81.1 (#118) Co-authored-by: Yuqi Huang <yuuqih@amazon.com> * feat: conditionally hide UI elements based on user settings (#119) - Hide data channel input when ingest media is enabled - Hide local video view when send video is disabled Co-authored-by: Yuqi Huang <yuuqih@amazon.com> * updated version from 1.1.0 to 1.2.0 (#120) Co-authored-by: Yuqi Huang <yuuqih@amazon.com> * update ci.yml to add develop branch to the ci target (#122) Co-authored-by: Yuqi Huang <yuuqih@amazon.com> * Updated Readme.md for WebRTC ingestion support (#121) Co-authored-by: Yuqi Huang <yuuqih@amazon.com> --------- Co-authored-by: Yuqi Huang <yuuqih@amazon.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.