Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Apply Regex precompile to TagParser #9573

Closed
emeroad opened this issue Dec 30, 2022 · 0 comments · Fixed by #9600
Closed

Apply Regex precompile to TagParser #9573

emeroad opened this issue Dec 30, 2022 · 0 comments · Fixed by #9600
Assignees
Milestone

Comments

@emeroad
Copy link
Member

emeroad commented Dec 30, 2022

TagParser.java

    private static String[] parseMultiValueFieldList(String string) {
        return string.replaceAll("[\\[\\]\"]", "").split(",");
    }

    public static String toTagStrings(String jsonTagStrings) {
         return jsonTagStrings.replaceAll("[{}\"]", "");
    }
@emeroad emeroad added this to the 2.5.1 milestone Dec 30, 2022
donghun-cho added a commit to donghun-cho/pinpoint that referenced this issue Jan 4, 2023
donghun-cho added a commit to donghun-cho/pinpoint that referenced this issue Jan 4, 2023
donghun-cho added a commit to donghun-cho/pinpoint that referenced this issue Jan 5, 2023
donghun-cho added a commit to donghun-cho/pinpoint that referenced this issue Jan 5, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants