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

NegotiateStream_Tcp_With_Upn test throw platform not supported exception #1260

Closed
hongdai opened this issue Jun 7, 2016 · 3 comments
Closed
Assignees
Milestone

Comments

@hongdai
Copy link
Contributor

hongdai commented Jun 7, 2016

It throws as we currently do not support UpnObjectComparer:Equals in ClaimComparer.cs

if SUPPORTS_WINDOWSIDENTITY

    private class UpnObjectComparer : IEqualityComparer
    {
        bool IEqualityComparer.Equals(object obj1, object obj2)
        {
            throw ExceptionHelper.PlatformNotSupported();
        }
@zhenlan zhenlan added this to the Future milestone Jun 7, 2016
@zhenlan zhenlan modified the milestones: 1.1.0, Future Jun 27, 2016
@roncain
Copy link
Contributor

roncain commented Sep 1, 2016

@hongdai -- can you provide instructions how you ran this and the stack trace? I see from code inspection what you mean, but it would be good to have repro steps to know when it is fixed. Also, is this related to #1235?

Also, FYI, this test was renamed to NegotiateStream_Tcp_With_SPN by me because it was using SPN, not UPN. Either the test name was wrong or the test itself was wrong. Do you know which? This is why repro steps would be valuable.

@roncain
Copy link
Contributor

roncain commented Sep 13, 2016

I think there is still some confusion around this issue. It claims to test UPN, but on examination was using SPN. I changed the name to match what the test did but left the test code unmodified. For this reason, I don't understand how the title of this issue points to the failure being related to UPN.

@hongdai -- could you clarify? Are we missing test coverage with TCP and UPN?

I believe PR #1502 should fix UPN issues at the same time it fixes #1454. But the test as written is not UPN.

@zhenlan zhenlan modified the milestones: 2016.10, 1.1.0 Oct 4, 2016
@hongdai
Copy link
Contributor Author

hongdai commented Oct 6, 2016

I have verified this issue is fixed by PR #1502 and enable the test disabled by this issue.

@hongdai hongdai closed this as completed Oct 6, 2016
@zhenlan zhenlan modified the milestones: 2016.10, 1.1.0 Oct 27, 2016
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

No branches or pull requests

3 participants