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

Adding websockets to the core script #232

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bocklucas
Copy link

Ft Add Websockets to Core Script

Changes

  • Added websocket publishing of events to the core script. By default websockets are not enabled
  • Added 3 additional optional args to the __main__.py file
    • websocket-host, default="localhost"
    • websocket-port, default=8675
    • enable-event-websockets, default=False

QA

  • Run the updated section in the README to verify the above changes
  • Run other sections to ensure nothing broke and that the server will function without the websockets being enabled

@bocklucas
Copy link
Author

CC @synesthesiam

@florian-asche
Copy link

what are you using the websocket for?

@bocklucas
Copy link
Author

@florian-asche great question, I was wanting to create a webpage that could be visited that would subscribe to events and react, say display images or render a 3d model, or really anything, as long as you're listening to satellite events

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants