Skip to content

Add AuthenticationDisplayName to HttpSys #21410

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

Merged
merged 4 commits into from
May 2, 2020

Conversation

Kahbazi
Copy link
Member

@Kahbazi Kahbazi commented May 1, 2020

Fixes #21281

/// <summary>
/// Sets the display name shown to users on login pages. The default is null.
/// </summary>
public string AuthenticationDisplayName { get; set; }
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

I couldn't generate reference assembly with scripts, so I just edit the file manually. I hope it works.

Copy link
Member

Choose a reason for hiding this comment

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

What happened when you tried?

@Tratcher
Copy link
Member

Tratcher commented May 1, 2020

Wow, that was fast. Did you set up notifications for HelpWanted issues somehow?

@Tratcher Tratcher self-assigned this May 1, 2020
@Kahbazi
Copy link
Member Author

Kahbazi commented May 1, 2020

No, I it was a normal subscription. I was just online ... and act fast 😁

@Tratcher Tratcher added this to the 5.0.0-preview5 milestone May 1, 2020
@ghost
Copy link

ghost commented May 1, 2020

Hello @Tratcher!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@Tratcher
Copy link
Member

Tratcher commented May 1, 2020

So close:

-        public bool AuthenticationDisplayName { [System.Runtime.CompilerServices.CompilerGeneratedAttribute] get { throw null; } [System.Runtime.CompilerServices.CompilerGeneratedAttribute] set { } }
+        public string AuthenticationDisplayName { [System.Runtime.CompilerServices.CompilerGeneratedAttribute] get { throw null; } [System.Runtime.CompilerServices.CompilerGeneratedAttribute] set { } }

@ghost ghost removed the auto-merge label May 1, 2020
@Tratcher Tratcher merged commit a4c4526 into dotnet:master May 2, 2020
@Tratcher
Copy link
Member

Tratcher commented May 2, 2020

Thanks

@Kahbazi Kahbazi deleted the kahbazi/httpsysAuth branch May 2, 2020 16:12
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HttpSys does not set DisplayName for Authentication Scheme Options
3 participants