Skip to content
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

Core Networking optimizations, fixes, reformating code and descriptive logging #50

Merged
merged 6 commits into from
Nov 27, 2020

Conversation

ShootingKing-AM
Copy link
Member

@ShootingKing-AM ShootingKing-AM commented Nov 25, 2020

Possible fix for #43 and #49

Added more descriptive debug logging in Networking core
…tion

1. Added an extra customizable option "connection_timeout" def: 5secs, to json, which can be adjusted as per user requirement.
2. Remove inconsistent bomb-based waiting method for AddtionalData, and replace with simple sequential timed wait.
1. Fixed Mobile not able to connect to PC using PCIP. (Now Tethering and Mobile/PC/Any Hotsports can be used by manually enterning the PCIP into `Settings` Page.
2. Added 10ms Delay in between UDP Announces to avoid network flood.
3. Improved descriptive logging.
@ShootingKing-AM ShootingKing-AM added enhancement feature improvement or request feature New feature fix Code fixes labels Nov 26, 2020
Fixing Autodiscovery not reaching all network interfaces. Now PVR will get all network interfaces and broadcast to all interfaces adding support for **Hotspot and Tethering streaming**.

And minor formating fixes.
Enhancing log readability by preventing wordwrap on log dump in Settings.kt activity
@ShootingKing-AM ShootingKing-AM self-assigned this Nov 27, 2020
@ShootingKing-AM ShootingKing-AM merged commit 0b24f93 into master Nov 27, 2020
@ShootingKing-AM ShootingKing-AM deleted the core-networking branch November 27, 2020 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature improvement or request feature New feature fix Code fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant