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

Fix UpnEndpointIdentity on UWP #1502

Merged
merged 2 commits into from
Sep 16, 2016
Merged

Fix UpnEndpointIdentity on UWP #1502

merged 2 commits into from
Sep 16, 2016

Conversation

mconnew
Copy link
Member

@mconnew mconnew commented Sep 2, 2016

Fixes #1454

// If WindowsIdentity isn't supported, then we can't
// retrieve the SecurityIdentifier's to compare so
// must return false
return false;
Copy link
Member

Choose a reason for hiding this comment

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

I guess this is going to be a necessary evil in order for UWP to work... we however need to clearly show this in the release notes.

@iamjasonp
Copy link
Member

LGTM

@@ -258,12 +255,36 @@ int IEqualityComparer.GetHashCode(object obj)
}
}

#if SUPPORTS_WINDOWSIDENTITY
//#if SUPPORTS_WINDOWSIDENTITY
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this line?

@roncain
Copy link
Contributor

roncain commented Sep 13, 2016

LGTM.
@mconnew -- I think when this is merged we can remove the Issue(10) we have on a few tests. It was accidentally unconditional, when it really only applied to UWP. Don't alter this PR -- just help me remember when you merge, and I'll make the change and verify it.

@mconnew mconnew merged commit b3a031c into dotnet:master Sep 16, 2016
roncain added a commit to roncain/wcf that referenced this pull request Sep 19, 2016
PR dotnet#1502 fixed issue dotnet#10 by implementing UpnEndpointIdentity.
But 2 tests were still disabled by issue 10, so this PR just
re-enables those tests.  Both require manual setup to run, so
we will not see them pass or fail in CI or VSO.
roncain added a commit to roncain/wcf that referenced this pull request Sep 19, 2016
PR dotnet#1502 fixed issue dotnet#10 by implementing UpnEndpointIdentity.
But 2 tests were still disabled by issue 10, so this PR just
re-enables those tests.  Both require manual setup to run, so
we will not see them pass or fail in CI or VSO.

These tests still fail in UWP and so remain disabled there.
mconnew added a commit to mconnew/wcf that referenced this pull request Sep 21, 2016
Fix UpnEndpointIdentity on UWP

Re-enable tests disabled by issue 10, except for NET Native.

PR dotnet#1502 fixed issue dotnet#10 by implementing UpnEndpointIdentity.
But 2 tests were still disabled by issue 10, so this PR just
re-enables those tests.  Both require manual setup to run, so
we will not see them pass or fail in CI or VSO.

These tests still fail in UWP and so remain disabled there.
mconnew added a commit to mconnew/wcf that referenced this pull request Sep 22, 2016
Fix UpnEndpointIdentity on UWP

Re-enable tests disabled by issue 10, except for NET Native.

PR dotnet#1502 fixed issue dotnet#10 by implementing UpnEndpointIdentity.
But 2 tests were still disabled by issue 10, so this PR just
re-enables those tests.  Both require manual setup to run, so
we will not see them pass or fail in CI or VSO.

These tests still fail in UWP and so remain disabled there.
@mconnew mconnew deleted the Issue1454 branch September 27, 2019 17:50
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.

5 participants