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

fourCcList definition should provide for a wildcard #15

Closed
zenomt opened this issue May 26, 2023 · 4 comments
Closed

fourCcList definition should provide for a wildcard #15

zenomt opened this issue May 26, 2023 · 4 comments

Comments

@zenomt
Copy link

zenomt commented May 26, 2023

in traditional RTMP, clients that are capable of receiving any codec (like recorders or forwarders) can just set the videoCodecs and audioCodecs fields to 0xffff to enable all possible code points. however, the number of possible future fourCCs is way too large to list them all. to accommodate clients that can handle any codec, i propose defining a special value to include in the fourCcList codec list (or its replacement(s)) to indicate "any/all codecs".

i further propose this special value to be the one-character string "*" (ASTERISK). being one character long it can never conflict with any real fourCC, and * is already commonly used to mean "wildcard".

@veovera
Copy link
Owner

veovera commented May 28, 2023

Interesting idea. It might behove to expand the special style even further to help solve #10. Please see my comments there.

@zenomt
Copy link
Author

zenomt commented May 28, 2023

i followed up in #10 as well, but for completeness here: i don't think we should add delimiters and ordering to fourCcList, but i do like the idea of wildcard strings for "any codec", "any audio codec", and "any video codec". my recommendations are:

  • *: "any codec"
  • audio/*: "any audio codec"
  • video/*: "any video codec"

@zenomt
Copy link
Author

zenomt commented May 28, 2023

note though that "any audio codec" and "any video codec" would be solved with * by having separate lists for audio and video, and the separate wildcard strings are only needed for the unified fourCcList case.

@veovera
Copy link
Owner

veovera commented May 16, 2024

Fixed, please see https://veovera.org/docs/enhanced/enhanced-rtmp-v2.

@veovera veovera closed this as completed May 16, 2024
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

No branches or pull requests

2 participants