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

Local connectivity #61

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

Local connectivity #61

wants to merge 3 commits into from

Conversation

Sheraff
Copy link
Owner

@Sheraff Sheraff commented Apr 30, 2023

Try to use intranet when client and server are on the same network.

  • find IP of server
  • auth / security kinda works: no actual auth token, but an ID that requires auth to obtain, and then can be used by a single IP, within the same network which seems like enough
  • intercept every HTTP request and add required parameters
  • polyfill secure headers: dpr and viewport are required for image generation, but aren't passed on insecure connections, so we just store it when obtaining the ID and re-use based on matched ID
  • upload of files => doesn't work because I either can't modify the Request url, or if I do I lose the FormData duplex
  • switch websocket connections => doesn't work though I haven't investigated too much, i just don't get any connection request and the client hangs forever
  • stream audio files => barely works, it doesn't seem to handle streaming, but does end up sending the entire file somehow (no idea how, in the Network tab, every request is either failed or canceled or took forever and has weird headers / no data...)

Maybe a better alternative to all of this mess would be to limit the scope to

  • streaming audio
  • uploading audio
  • maybe if we're feeling frisky, the images as well

This is what takes most of the bandwidth anyway

@Sheraff Sheraff added enhancement New feature or request help wanted Extra attention is needed labels Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant