Add RFC: Protocol API

Author: tytan652

text/0045-protocols-in-outputs-api.md

@@ -0,0 +1,110 @@
+***This RFC is dependent on [RFC 39](https://github.com/tytan652/rfcs/blob/service_overhaul/text/0039-service-overhaul.md).***
+
+# Summary
+
+- Add notion of protocol in service outputs
+- Those outputs are registered with a list of compatible codecs
+- Show only stream encoders compatible with the selected outputs
+- Add as byproduct the possibility to add plugin with new service output with a new protocol (useless without RFC 39)
+
+# Motivation
+
+Create a better management of outputs, encoders and services with a protocol centered approach.
+
+Byproduct: Add also the possibility to support third-party plugin which add protocol support with his own output.
+
+*Services addition is a part of [RFC 39](https://github.com/tytan652/rfcs/blob/service_overhaul/text/0039-service-overhaul.md).*
+
+# Design
+*This RFC does not consider that because a protocol support a encoder it should be implemented in the main project.*
+
+## Outputs API
+**This only concern outputs with the flag `OBS_OUTPUT_SERVICE`.**
+
+Adding to the API those fields:
+- `protocol`: protocols "official" acronym (ex: RTMP, WebRTC)
+- `protocol_prefix` (optional): for custom service detection (eg `rtmp://`, `srt://`)
+
+A output only support one protocol like an encoder support only one codec.
+
+Adding to the API those functions:
+- `bool obs_output_find_protocol(const char*)`: return true if an output with the protocol is registered
+- `const char *obs_output_find_output_with_protocol(const char *)` return an output corresponding to the protocol (the first match if various)
+- a way to enum prefixes and/or a way to find protocol based on the prefix so `obs_output_enum_protocol_prefix` and/or `obs_output_find_protocol_with_prefix`
+
+Those function will be ignore by obs-scripting (Python and Lua).
+
+**This may concern any outputs.**
+Adding to the API those fields:
+- Protocols like SRT are codec agnostic so:
+  - `video_codec_agnostic` (boolean, false by default if not set): if true, disable the need of `encoded_video_codecs`
+  - `audio_codec_agnostic` (boolean, false by default if not set): if true, this disable the of `encoded_audio_codecs`
+  - Or just consider `NULL` as codec agnostic ?
+
+### About Protocols
+`encoded_video_codecs` and `encoded_audio_codecs` shall contain any codec compatible with the protocol following their specification. Except RTMP, it will only support H.264 and AAC for now.
+
+It does not mean that services are compatible with all the codecs present in the specs. And neither mean that OBS support those codecs.
+
+Protocol with variation like RTMP may need to have an output per variation to avoid using RTMPS services if the build of OBS does not support it.
+
+The actually supported protocol with their specification documents :
+- RTMP (https://wwwimages2.adobe.com/content/dam/acom/en/devnet/rtmp/pdf/rtmp_specification_1.0.pdf)
+- RTMPS (same as RTMP)
+- HLS (https://datatracker.ietf.org/doc/html/rfc8216)
+- SRT (https://github.com/Haivision/srt/files/2489142/SRT_Protocol_TechnicalOverview_DRAFT_2018-10-17.pdf ; TLDR: codec agnostic)
+- FTL (no official one found except this https://github.com/microsoft/ftl-sdk/blob/master/README.md)
+
+### Maybe
+
+`encoded_video_codecs` and `encoded_audio_codecs` rather than being string to split, could be replaced an array.
+
+## Services API
+Adding to the API the field `protocol` which match the Outputs API one.
+
+With RFC 39, OBS could prevent showing services where the asked protocol is not in any registered service output.
+And also make services able to not show unsupported protocol if multi-protocol.
+
+We shall consider that if a service handle a protocol, it doens't mean it can handle all the compatible codecs.
+So if a service can't handle all the compatible codec, adding a list of supported codec to a service related to his protocol shall be considered.
+
+Adding to the API those fields:
+- `supported_video_codecs` (optional): sometime stream services don't support all the codec of the specification
+- `supported_audio_codecs` (optional): sometime stream services don't support all the codec of the specification
+
+`rtmp-services` will be modified to obtain this behavior beforehand with only H.264, AAC and Opus as compatible codecs.
+
+## UI
+Changing protocol/output through service change will update the Output page with only compatible encoders.
+
+Compatible encoders are based on what service can handle and if they specify nothing, any codec accepted by the protocol will be shown.
+
+If the service ask for codec not compatible with the protocol, it will be ignored and written in the log as error.
+
+It also may reset encoder settings if the actual encoder was not compatible.
+
+The user may need to be warn about the fact that the encoder was not compatible and he need to setup again his encoder.
+
+## What if there is various registered outputs for one protocol ?
+It could happen that a plugin register a output for a already registered protocol, so we need to manage this possibility.
+
+- Add `preferred_output` to the Services API to use what OBS already provide for some protocols with maybe some hardcode in his getter or maybe hardcode the default output per protocol in the UI.
+- Add `obs_output_find_outputs_with_protocol(const char *)` return all outputs corresponding to the protocol to the Outputs API.
+- Make the user able also to choose an output if various providing the same protocol are registered (e.g FFmpeg SRT and "Native" SRT) as an advanced options.
+- Deprecate `obs_service_get_output_type` because it got replaced.
+
+## Other possible addition
+
+- Make the user able also to choose an audio encoder if various as an advanced options.
+
+## Codec naming
+For codec naming, we try to get the closiest of an official typography even inside a logo.
+
+- H.264/AVC will be named/changed to `H.264`
+- H.265/HEVC will be named `HEVC` since this name is used rather than H.265, only added if adding the fact that a protocol is compatible with it will not cause legal issue.
+- AAC we keep `AAC`.
+- Opus we keep `opus`.
+
+# Benefit
+- FTL, if deprecated by OBS and still used by stream services could be separated in a plugin providing the protocol and the output and also service with [RFC 39
+](https://github.com/tytan652/rfcs/blob/service_overhaul/text/0039-service-overhaul.md)