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

Test PR for #68366 #68700

Closed
wants to merge 1 commit into from
Closed

Test PR for #68366 #68700

wants to merge 1 commit into from

Conversation

filipnavara
Copy link
Member

No description provided.

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Apr 29, 2022
@dotnet-issue-labeler dotnet-issue-labeler bot added area-System.Net.Security and removed community-contribution Indicates that the PR has been added by a community member labels Apr 29, 2022
@ghost
Copy link

ghost commented Apr 29, 2022

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

Issue Details

null

Author: filipnavara
Assignees: -
Labels:

area-System.Net.Security

Milestone: -

@filipnavara
Copy link
Member Author

/azp run runtime-libraries enterprise-linux

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 68700 in repo dotnet/runtime

@vcsjones
Copy link
Member

/azp run runtime-libraries enterprise-linux

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@vcsjones
Copy link
Member

/azp list

@vcsjones
Copy link
Member

Hm. Not sure where the enterprise-linux pipeline went. @wfurt?

@wfurt
Copy link
Member

wfurt commented Apr 29, 2022

It is disabled now as it was constantly failing @vcsjones. I can do local test and if that looks promising we can take another step.

While I don't mind the fix in general, I'm not sure what it is needed now and was not needed before @filipnavara.

@filipnavara
Copy link
Member Author

filipnavara commented Apr 29, 2022

I just wanted to see if the test will pass with this check removed. The Negotiate MIC was never checked before my last changes so either this check didn't work for a while, or it should fail with a different error.

@ghost ghost closed this May 29, 2022
@ghost
Copy link

ghost commented May 29, 2022

Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it.

@ghost ghost locked as resolved and limited conversation to collaborators Jun 29, 2022
@karelz karelz added this to the 7.0.0 milestone Jul 19, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants