Skip to content

Fix OpenSSL version check in GetAlpnSupport - #64023

Merged
bartonjs merged 1 commit into
mainfrom
fix_alpn_platformdetection
Jan 20, 2022
Merged

Fix OpenSSL version check in GetAlpnSupport#64023
bartonjs merged 1 commit into
mainfrom
fix_alpn_platformdetection

Conversation

@bartonjs

@bartonjsbartonjs commented Jan 19, 2022

Copy link
Copy Markdown
Member

The previous check failed 3.0.0 because the Minor was 0 and Build was 0.

It could probably be rewritten to be >= new Version(1, 0, 2), but that'd require more thinking.

Fixes#63624.

The previous check failed 3.0.0 because the Minor was 0 and Build was 0.
It could probably be rewritten to be `>= new Version(1, 0, 2)`, but that'd require more thinking.
@bartonjsbartonjs added area-System.Net.Security test-bug Problem in test source code (most likely) labels Jan 19, 2022
@bartonjs
bartonjs requested a review from wfurtJanuary 19, 2022 23:42
@ghostghost assigned bartonjsJan 19, 2022
@ghost

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/ncl, @vcsjones
See info in area-owners.md if you want to be subscribed.

Issue Details

The previous check failed 3.0.0 because the Minor was 0 and Build was 0.

It could probably be rewritten to be >= new Version(1, 0, 2), but that'd require more thinking.

Author:bartonjs
Assignees:-
Labels:

area-System.Net.Security, test bug

Milestone:-

@wfurtwfurt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.
thanks.

@bartonjs
bartonjs merged commit e4bfedd into mainJan 20, 2022
@bartonjs

Copy link
Copy Markdown
MemberAuthor

/backport to release/6.0

@bartonjs
bartonjs deleted the fix_alpn_platformdetection branch January 20, 2022 08:06
@github-actions

Copy link
Copy Markdown
Contributor

Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/1722287863

@ghostghost locked as resolved and limited conversation to collaborators Feb 19, 2022
@karelzkarelz added this to the 7.0.0 milestone Apr 8, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Net.Securitytest-bugProblem in test source code (most likely)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cryptography/security test failures on RHEL9/Fedora 36.

4 participants

@bartonjs@stephentoub@wfurt@karelz