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

Got wrong MachineName when using Exceptionless client v4.4.0 #233

Closed
witskeeper opened this issue Jul 24, 2020 · 8 comments
Closed

Got wrong MachineName when using Exceptionless client v4.4.0 #233

witskeeper opened this issue Jul 24, 2020 · 8 comments

Comments

@witskeeper
Copy link
Contributor

Got wrong MachineName when using Exceptionless client v4.4.0 with ASP.NET Core 3.1.x

Exceptionless v4.3.x works well.

@niemyjski
Copy link
Member

niemyjski commented Jul 24, 2020

What is the difference between process machine name and the environment machine name for you? Also, I'm wondering if we should be falling back to environment machine name if the process.MachineName is empty? (would it ever be empty). Looks like Process.MachineName will return the computer name where the process is running on . https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics.process.machinename?view=netcore-3.1 Makes me think we should be using the process instead of the actual machine name.

@witskeeper
Copy link
Contributor Author

this bug came from v4.4.0 and you can see the changes of code.
old version use process.MachineName when #elif !PORTABLE && !NETSTANDARD1_2

So , we can just use Environment.MachineName now.

DE422C88-80A7-4173-94B9-169D7000CCAD

@niemyjski
Copy link
Member

Thanks for the PR!

@witskeeper
Copy link
Contributor Author

witskeeper commented Jul 29, 2020

when will the new version be released ? @niemyjski

@niemyjski
Copy link
Member

We like to try and get a few changes out in each release, can you use a nightly build for now?

@yuefengkai
Copy link

If this issues is resolved, please notify me, thank you @niemyjski

@niemyjski
Copy link
Member

this issue is resolved and merged in, you can find the nuget packages on our nightly ci feed: https://f.feedz.io/exceptionless/exceptionless/nuget/index.json

@niemyjski
Copy link
Member

A new release is building right now: https://github.com/exceptionless/Exceptionless.Net/releases/tag/v4.4.1

yuefengkai pushed a commit to yuefengkai/serilog-sinks-exceptionless that referenced this issue Jul 31, 2020
niemyjski pushed a commit to exceptionless/serilog-sinks-exceptionless that referenced this issue Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants