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: Wrap routeableipv4 in try catch #6644

Merged
merged 2 commits into from Mar 28, 2024
Merged

Conversation

covert8
Copy link
Contributor

@covert8 covert8 commented Mar 22, 2024

Description

Wrap NetworkInterface.GetAllNetworkInterfaces in try catch to solve race condition on aarch64

Database Migration

NO

Issues Fixed or Closed by this PR

@covert8 covert8 force-pushed the wrap-ipv6-check branch 2 times, most recently from 123cf0b to 6c213ab Compare March 22, 2024 09:45
Copy link
Member

@markus101 markus101 left a comment

Choose a reason for hiding this comment

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

Should we log anything so people know what's going on without having to dig into code?

@covert8
Copy link
Contributor Author

covert8 commented Mar 24, 2024

I'm having some trouble finding how to include the logger when the class is being constructed, could you lend a hand?

And it would seem that the functions can't be static any more if we start logging.
@markus101

Copy link
Member

@markus101 markus101 left a comment

Choose a reason for hiding this comment

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

Converting these methods to non-static seems reasonable, shouldn't make too much of an impact over all.

src/NzbDrone.Core.Test/Framework/CoreTest.cs Outdated Show resolved Hide resolved
@covert8 covert8 force-pushed the wrap-ipv6-check branch 2 times, most recently from 86e98e1 to 6747fc4 Compare March 25, 2024 08:30
Co-authored-by: Mark McDowall <markus.mcd5@gmail.com>
Co-authored-by: Bogdan <mynameisbogdan@users.noreply.github.com>
@covert8 covert8 requested a review from markus101 March 27, 2024 09:00
@markus101 markus101 merged commit 060b789 into Sonarr:develop Mar 28, 2024
13 checks passed
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.

None yet

3 participants