imperva / imperva/trace4rs

rm regex

Open
#16 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
6
Forks
2
PR merge metrics
No merged PRs in 30d

Description

```rust
static RE: once_cell::sync::Lazy =
once_cell::sync::Lazy::new(|| regex::Regex::new(r#"\$ENV\{([\w][\w|\d|\.|_]*)\}"#).unwrap());
```

Didn't this get changed in log4rs? Or rather, wasn't the regex stuff removed? If so maybe we should consider removing it here as well, as a part of future work.

Contributor guide

Open the contributing guide

Research direction

Read the static RE declaration shown in the issue and compare the related change in log4rs. Identify whether the regex machinery is still needed; done means the obsolete regex-related code is removed without breaking trace4rs.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.