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

SRTP_AES256 is added according draft-ietf-avt-dtls-srtp-00 #580

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

keremcadirci
Copy link

Describe your changes

SRTP_AES256 is added according draft-ietf-avt-dtls-srtp-00
Both SRTP_AES256_CM_HMAC_SHA1_80 and SRTP_AES256_CM_HMAC_SHA1_32 are widely used in SRTP
Only new items are added to the "Enum" Class

How has this been tested?

Only new items are added to the "Enum" Class. I believe it has no effect

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have kept the patch limited to only change the parts related to the patch
  • This change requires a documentation update

See also Contributing Guidelines.

@peterdettman
Copy link
Collaborator

These profiles were removed in later drafts and do not appear in the published RFC 5764.

A quick search found only a single page of search results, and only a single github project that has the profiles (which it incorrectly documents as appearing in RFC 5764). Please justify your claim that they are widely used in SRTP.

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

Successfully merging this pull request may close these issues.

2 participants