Skip to content

Conversation

@dgeibi
Copy link
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 Enhancement Improvement to existing functionality label Jan 18, 2021
@henke37
Copy link
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
Member

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

@WizardCM
Copy link
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

Enhancement Improvement to existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants