Graylog2 / Graylog2/graylog2-server

Consolidate whitespace handling

Open
#20,983 0 comments 0 reactions 1 assignee Claimed by @dennisoelkers View on GitHub
feature triaged
Dominant language
Java
Stars
8.1k
Forks
1.1k
Avg merge
1d 20h
Merged PRs (30d)
217

Description

## What?

We have places in Graylog where we check/replace whitespace. In some of the occurrences (see link below) we check for a simple "space" char - which is not, what Java suggests as whitespace. If we allow a "space" in field names at some point, we should consolidate all whitespace checks to be the same, too.

https://github.com/Graylog2/graylog2-server/blob/0febd8f300578a238fbf1a8e8a89d2f3f2f46628/graylog2-server/src/main/java/org/graylog2/inputs/extractors/JsonExtractor.java#L129

https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/lang/Character.html#isWhitespace(char)

* Graylog Version: 6.2-SNAPSHOT

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.