trimming contentString

[frogcjn]

stringContent may contain a prefix new line ("\n") before the first "data:" separator.

What

When query with chat streaming API, the response may contain a prefix new line (\n) before the first data: separator.

Affected Areas

Chat Streaming APIs

[ingvarus-bc]

Hi, @frogcjn, thanks for your interest in the project and for your PR.
It seems like character trimming for each data component is already being done on line 63:

.map { $0.trimmingCharacters(in: .whitespacesAndNewlines) }

So, I am not exactly sure we need to do that twice. What do you think?

[frogcjn]

There are some cases there is "\n" before the "data:"， and nothing before "\n".

The first element is "\n" after separate by "data:".

After the line63 trimming, the first still exists and it becomes empty

then the first empty string will raise error when decoding JSON.

[ingvarus-bc]

Got you, thanks!

[sonarcloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud