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

[API] Crypto mode 'auto' implemented for listener. #2574

Merged

Conversation

maxsharabayko
Copy link
Collaborator

@maxsharabayko maxsharabayko commented Dec 7, 2022

For an SRT listener to accept any crypto mode (AES-GCM or AES-CTR) the SRTO_CRYPTOMODE=0 (Auto) can be used.

When SRTO_CRYPTOMODE=0 (Auto) is set on an SRT Caller or SRT Rendezvous the actual mode to be negotiated is AES-CTR.

This change does not require further HS changes on the wire, still allows to let the listener to accept any crypto mode.

Once a connection is established, the actual crypto mode is returned via the SRTO_CRYPTOMODE: 1 (AES-CTR) or 2 (AES-GCM).

@maxsharabayko maxsharabayko added Type: Enhancement Indicates new feature requests [API] Area: Changes in SRT library API labels Dec 7, 2022
@maxsharabayko maxsharabayko added this to the v1.6.0 milestone Dec 7, 2022
srtcore/crypto.cpp Outdated Show resolved Hide resolved
@maxsharabayko maxsharabayko force-pushed the develop/aead-crypto-auto branch 2 times, most recently from 3baba24 to 0416b96 Compare December 8, 2022 10:43
Caller or rendezvous still falls back to AES-CTR.
@codecov-commenter

This comment was marked as off-topic.

srtcore/core.cpp Outdated Show resolved Hide resolved
Copy link
Collaborator

@stevomatthews stevomatthews left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor edits.

Co-authored-by: stevomatthews <smatthews@haivision.com>
@maxsharabayko maxsharabayko merged commit 60d1237 into Haivision:master Dec 12, 2022
@maxsharabayko maxsharabayko deleted the develop/aead-crypto-auto branch December 12, 2022 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[API] Area: Changes in SRT library API Type: Enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants