bramp / bramp/ffmpeg-cli-wrapper

Allow user to replace all uses of STDOUT/STDERR for logging

オープン
#355 コメント 4 件 リアクション 1 件 担当者 0 名 GitHub で見る
enhancement
主要言語
Java
スター
1.9k
フォーク
424
平均マージ
5時間 35分
マージ済み PR(30日)
2

説明

Hello,

I deploy this library in an environment where the java processes stdout and stderr is discarded and there is no way for the developer
maintainer to see it.

I would like the ffmpeg errors to be captured without involving stderr as they are usefull to diagnose some errors.
To capture the errors you would need to either log them into java.util.logging or the slf4j logging facade.

I understand that you may not wish to include slf4j as a dependency.
I know that some people hate java.util.logging (and would prefer not to have to use it)

So my suggestion would be to have a global static variable that the user can set that determines where log lines get pumped to.
The default implementation would pump to stdout/stderr just like it currently does but I could then in my bootstrap of my application call
"FFmpegCommon.setLoggingSystem(FFmpegLoggingManager instance)" (just an example)

If I were to design this interface then I would give it only 1 method that looks like this:

void appendLog(Process ffmpegProc, boolean isStdErr, byte[] data, int off, int len);

If you do not want to hand in the process then please pass some unique identifier (aka process pid for example or a random UUID instead so I can tell appart error messages from multiple concurrent ffmpeg jobs)

I would outsource line breaks and all that thing to concrete implementations.
Currently I have to hack very deep into your library using inheritance to be able to accomplish this.

The default implementation that dumps everything to stdout/stderr could probably be as simple as this:

```java
public void appendLog(Process ffmpegProc, boolean isStdErr, byte[] data, int off, int len) {
PrintStream stream = isStdErr ? System.err : System.out;
try {
stream.write(data, off, len);
} catch(Exception e) {
//IGNORED
}
}
```

Sincerely
Alexander Schütz

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

まず、FFmpeg プロセスの出力を System.out と System.err に書き込むコードパスを、現在カスタマイズに必要な継承フックも含めて特定します。設計の入り口として、提案されている FFmpegCommon.setLoggingSystem と appendLog インターフェースを使用します。完了の条件は、デフォルトの動作を変更せずに呼び出し元が両方のストリームをリダイレクトできることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
cli, tooling
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。