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

System.UriFormatException on load (Mono) #1966

Merged
merged 1 commit into from
Feb 15, 2017
Merged

System.UriFormatException on load (Mono) #1966

merged 1 commit into from
Feb 15, 2017

Conversation

JustArchi
Copy link
Contributor

@JustArchi JustArchi commented Feb 14, 2017

Fixes #1965

This change is Reviewable

@JustArchi
Copy link
Contributor Author

I'm pretty sure that Travis failure is a mere coincidence of broken Mono, as the build got stalled. Please re-run it if needed.

@304NotModified
Copy link
Member

Retied travis but still an issue?

@304NotModified
Copy link
Member

Mmmmm should indeed not trigger these issues. Will retry again

@304NotModified
Copy link
Member

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@304NotModified 304NotModified changed the title Fixes #1965 System.UriFormatException on load (Mono) Feb 15, 2017
@304NotModified 304NotModified added the bug Bug report / Bug fix label Feb 15, 2017
@304NotModified 304NotModified added this to the 4.4.3 milestone Feb 15, 2017
@304NotModified 304NotModified merged commit c277ecc into NLog:master Feb 15, 2017
@304NotModified
Copy link
Member

Unfortunately this needs to be ported to NLog 5 (coreCLR branch which is behind - still merging 4.4.2 into it) see #1944

@JustArchi
Copy link
Contributor Author

I sent a PR for CoreCLR in #1967 - thanks a lot for merging the bugfix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report / Bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants