Skip to content

NamedPipeServerStream.Unix: GetImpersonationUserName tweaks. - #88125

Merged
stephentoub merged 2 commits into
dotnet:mainfrom
tmds:nps_user
Jun 29, 2023
Merged

stephentoub merged 2 commits into
dotnet:mainfrom
tmds:nps_user

Conversation

@tmds

@tmds tmds commented Jun 28, 2023

Copy link
Copy Markdown
Member
  • When the user name is not found, return an empty string like Environment.UserName.
  • Add EINTR handling.

When the user name is not found, return an empty string like Environment.UserName.

Add EINTR handling.
@ghost ghost added area-System.Net community-contribution Indicates that the PR has been added by a community member labels Jun 28, 2023
@ghost

ghost commented Jun 28, 2023

Copy link
Copy Markdown

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

Issue Details
  • When the user name is not found, return an empty string like Environment.UserName.
  • Add EINTR handling.
Author: tmds
Assignees: -
Labels:

area-System.Net

Milestone: -

@tmds
tmds requested review from antonfirsov and stephentoub June 28, 2023 09:36
@tmds

tmds commented Jun 28, 2023

Copy link
Copy Markdown
Member Author

I've updated the PR to avoid the duplicate native functions that wrap getpwuid_r.

@stephentoub
stephentoub merged commit f99ebea into dotnet:main Jun 29, 2023
@karelz karelz added this to the 8.0.0 milestone Jul 3, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Aug 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Net community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants