Skip to content

Conversation

@rvagg
Copy link
Member

@rvaggrvagg commented Dec 4, 2015

  • Include reference to CVE-2015-8027
  • Fix "socket may no longer have a socket" reference
  • Expand on non-existent parser causing the error
  • Soften language for CVE-2015-3193 as impact may not be as great as
    expected, also trim to match v4.2.3 language
  • Clarify that CVE-2015-3194 affects TLS servers using client
    certificate authentication
  • Include reference to CVE-2015-6764

* Include reference to CVE-2015-8027 * Fix "socket may no longer have a socket" reference * Expand on non-existent parser causing the error * Soften language for CVE-2015-3193 as impact may not be as great as expected, also trim to match v4.2.3 language * Clarify that CVE-2015-3194 affects TLS servers using _client certificate authentication_ * Include reference to CVE-2015-6764
@JungMinuJungMinu added the doc Issues and PRs related to the documentations. label Dec 4, 2015
Copy link
Member

Choose a reason for hiding this comment

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

I speculate that attempts trigger -> attempts to trigger
(sorry If I'm wrong)

@JungMinu
Copy link
Member

LGTM with one small comment :)

@cjihrig
Copy link
Contributor

LGTM

1 similar comment
@jasnell
Copy link
Member

LGTM

rvagg added a commit that referenced this pull request Dec 5, 2015
* Include reference to CVE-2015-8027 * Fix "socket may no longer have a socket" reference * Expand on non-existent parser causing the error * Soften language for CVE-2015-3193 as impact may not be as great as expected, also trim to match v4.2.3 language * Clarify that CVE-2015-3194 affects TLS servers using _client certificate authentication_ * Include reference to CVE-2015-6764 PR-URL: #4156 Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
@rvaggrvagg closed this Dec 5, 2015
@rvaggrvagg deleted the v5.1.1-release-notes-update branch December 5, 2015 04:30
@rvagg
Copy link
MemberAuthor

rvagg commented Dec 5, 2015

thanks, fixed and landed @ 6c16c40

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@rvagg@JungMinu@cjihrig@jasnell