Skip to content

Register obsstudio URL handler for windows#4089

Closed
dgeibi wants to merge 1 commit intoobsproject:masterfrom
NiMO-TV:feat/obs-protocol
Closed

Register obsstudio URL handler for windows#4089
dgeibi wants to merge 1 commit intoobsproject:masterfrom
NiMO-TV:feat/obs-protocol

Conversation

@dgeibi
Copy link
Copy Markdown
Contributor

@dgeibi dgeibi commented Jan 18, 2021

Description

Register OBS to obsstudio://

Motivation and Context

Make it possible to open OBS from websites.

How Has This Been Tested?

Make nsis installers and install OBS. Create a html file with <a href="obsstudio://">Open OBS</a> and Click "Open OBS"

Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@WizardCM WizardCM added the kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. label Jan 18, 2021
@henke37
Copy link
Copy Markdown
Contributor

henke37 commented Jan 18, 2021

Protocol handlers have a bit of a history of excessive trust. Have you considered abuse potential? For example, does this offer the ability to pass command line arguments that aren't internet safe?

@WizardCM
Copy link
Copy Markdown
Member

It may be worth reading the RFC for a URI handler obsproject/rfcs#10

@WizardCM
Copy link
Copy Markdown
Member

Closing this for now as there still seems to be some discussion to be had on the RFC.

@WizardCM WizardCM closed this Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement Enhancements are not bugs or new features but can improve usability or performance.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants