スタッフログの大きな投稿がExcel CSV形式を破損させる

貢献 バグ
1

スタッフは /admin/logs/staff_action_logs からスタッフログをエクスポートして完全な履歴を確認できます。管理 UI 内ではログの一部のみが表示されるためです。特定のログには詳細にカンマが含まれることがあり、それが CSV の書式を崩す原因となります。

「いいね！」 8
2

Can commas be escaped in this output @vinothkannans?

「いいね！」 4
4

No, the commas are not breaking the CSV file. Maximum number of characters allowed in a single cell of MS Excel sheet is 32,767. If a column exceeded the characters limit then it’s breaking the CSV format. In our case details column having more characters in some post_edit staff actions. If I open the same CSV file in Google Sheets then it renders all the columns correctly.

If needed I’ll limit the maximum number of characters for details column to ~30,000 in the export unconditionally.

「いいね！」 8
5

30k chars still seems like a lot. Do we need that many characters in a cell? If so, why?

6

It’s due to giant posts. For example, many of our internal runbooks tend to be quite long. Likely only the post_edit staff action as Vinoth noted would have this many characters.

7

Also we storing both old and new post raws for post_edit staff actions in discourse/app/services/staff_action_logger.rb at 4383afb769d97bc9724d5448c0583b14c39782d0 · discourse/discourse · GitHub.

8

We should aggressively truncate this, I don’t see the point of creating such bulk in the database over something so trivial.

「いいね！」 5
9

Okay, I’ll try to improve this by storing only the edit differences in the raw text.

「いいね！」 4
10

こちらはクローズしましょう。シンプルに、セルを30kに制限してからクローズします。デルタ関連の変更は非常に複雑です。

「いいね！」 4
15

これを再オープンします。何らかの修正を導入する必要があります。巨大な投稿がフォーマットを壊していて、エクスポートされたログを確認することがほぼ不可能です。

「いいね！」 1
22

おそらく、ここで最も簡単な修正は次のとおりです。

> '"' + 'post "body" '.gsub('"', '""') + '"'
=> "\"post \"\"body\"\" \"\""

これはCSVで許可されており、機能するはずだと思います。