-
Notifications
You must be signed in to change notification settings - Fork 237
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
Apply the eslint fixes (from Component Governance Checks) #1596
Conversation
MSNev
commented
Jun 29, 2021
- Update ts-plugin dependency version
IChannelControls, hasWindow, hasDocument, isReactNative, doPerf, IDiagnosticLogger, INotificationManager, objForEachKey, proxyAssign, | ||
import { | ||
IConfiguration, AppInsightsCore, IAppInsightsCore, LoggingSeverity, _InternalMessageId, ITelemetryItem, ICustomProperties, | ||
IChannelControls, hasWindow, hasDocument, isReactNative, doPerf, IDiagnosticLogger, INotificationManager, objForEachKey, proxyAssign, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are just mixed whitespace issues (had spaces and tabs)
@@ -81,7 +81,7 @@ export class LogEntry { | |||
ms = "0" + ms; | |||
} | |||
|
|||
ms = ms.replace(/(.)(\d{3})+$/g, '$1.$2s') | |||
ms = ms.replace(/(.)(\d{3}){1}$/g, '$1.$2s') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The regex changes are security related and a little more complicated and the changes are 2 fold
- Unnecessary escaping (additional \ when not needed), this could allow someone to "fiddle" with the resulting regex (in memory) to change it's behavior
- When using * and +, depending on the input string unconstrained matching can cause the regex to use excessive memory, cpu or just fail.
ba948f8
to
2f6184b
Compare
- Update minimum dependency versions to address installation
2f6184b
to
095d6f1
Compare