Skip to content

Temporarily fix VB App Framework Logging. #7590

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.

Sign up for GitHub

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
KlausLoeffelmann merged 14 commits into from
Aug 15, 2022
Merged

Temporarily fix VB App Framework Logging. #7590

Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
6f722cf
Draft
dreddy-work Jul 1, 2022
1af2b94
Draft chnages for risk assesment.
dreddy-work Jul 20, 2022
4b07447
Introduce runtime-config settigns for opt-in or opt-out of a breaking…
dreddy-work Aug 10, 2022
b296123
Merge branch 'main' into dev/dreddy/fix7251
dreddy-work Aug 10, 2022
8117534
Address review feedback.
dreddy-work Aug 11, 2022
96bfd93
Adress further feedback
dreddy-work Aug 12, 2022
d0746ba
Fix test and address review comments.
dreddy-work Aug 12, 2022
22ce594
Test skip that was accidentally removed.
dreddy-work Aug 12, 2022
6651e56
Merge branch 'main' into FixVbLogging
KlausLoeffelmann Aug 14, 2022
3931bea
Patch Microsoft.VisualBasic.Logging.Log to unblock failing tests.
KlausLoeffelmann Aug 14, 2022
7dfca3d
Copy LocalAppContextSwitches functionality to Microsoft.VisualBasic.F…
KlausLoeffelmann Aug 15, 2022
68af597
Revert quirking, disable config file logging for My.Logging.Log.
KlausLoeffelmann Aug 15, 2022
b6f08db
Reset file.
KlausLoeffelmann Aug 15, 2022
bc0a179
Remove redundant Imports statement.
KlausLoeffelmann Aug 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 3 additions & 11 deletions src/Microsoft.VisualBasic.Forms/src/Microsoft/VisualBasic/Logging/Log.vb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -163,20 +163,12 @@ Namespace Microsoft.VisualBasic.Logging
If _listenerAttributes Is Nothing Then
_listenerAttributes = Attributes
End If
Return _hasBeenInitializedFromConfigFile
Copy link
Copy Markdown
Contributor

@Tanya-Solyanik Tanya-Solyanik Aug 15, 2022

Choose a reason for hiding this comment

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

I would had returned false here for the RC and would have worked on the proper fix, i.e. how to get if this was initialized from the config file.

Copy link
Copy Markdown
Member Author

@KlausLoeffelmann KlausLoeffelmann Aug 15, 2022

Choose a reason for hiding this comment

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

I thought about this first. But then I thought, there is a good chance it will stay like this, and then we could have it right away.

Copy link
Copy Markdown
Contributor

@Tanya-Solyanik Tanya-Solyanik Aug 15, 2022

Choose a reason for hiding this comment

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

there is a good chance it will stay like this,

Maybe runtime can add a protected property for us to look up?


' TODO: This is a tempory fix, which will break configuring logging via file for the time being. See: https://github.com/dotnet/winforms/pull/7590
Copy link
Copy Markdown
Contributor

@Tanya-Solyanik Tanya-Solyanik Aug 16, 2022

Choose a reason for hiding this comment

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

Did you open a follow up bug to get this fixed?

Return False
End Get
End Property

''' <summary>
''' Overriding this function is the trick that tells us whether this trace source was configured
''' from a config file or not. It only gets called if a config file was found.
''' </summary>
Protected Overrides Function GetSupportedAttributes() As String()
_hasBeenInitializedFromConfigFile = True
Return MyBase.GetSupportedAttributes()
End Function

Private _hasBeenInitializedFromConfigFile As Boolean 'True if we this TraceSource is initialized from a config file. False if somebody just news one up.
Private _listenerAttributes As StringDictionary

End Class
Expand Down