Skip to content

[logger] eliminate expensive regex #31061

Open
@xirzec

Description

@xirzec

This was flagged by an automated tool as a bad code pattern:

skippedNamespaces.push(new RegExp(`^${ns.substr(1)}$`));

and

enabledNamespaces.push(new RegExp(`^${ns}$`));

Perhaps we can clean this code up or consider taking a dependency on debug directly now that we aren't trying to support IE11...

Metadata

Metadata

Assignees

Labels

Azure.CoreClientThis issue points to a problem in the data-plane of the library.EngSysThis issue is impacting the engineering system.needs-team-attentionWorkflow: This issue needs attention from Azure service team or SDK team

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions