Tomo-9925 / cnet

Controlling and logging communication of process in Docker container
2 stars 0 forks source link

Feature/fix log and errors #17

Closed Tomo-9925 closed 3 years ago

Tomo-9925 commented 3 years ago

変更内容

関連Issue

ログ出力が正常でない #11

Tomo-9925 commented 3 years ago

やっとエラーが止まった…!

masibw commented 3 years ago

修正ありがとうございます マージする時はsquashしていただけるとありがたいです

まだコード見れてないのですがbuild通らなくないですか?

Tomo-9925 commented 3 years ago

まだコード見れてないのですがbuild通らなくないですか?

検証サーバだけ動く感じですかね…?

~/g/g/T/c/c/cnet feature/fix_log_and_errors k992077@k992077_sotsuken
❯ go build

~/g/g/T/c/c/cnet feature/fix_log_and_errors k992077@k992077_sotsuken
❯ echo $status
0
masibw commented 3 years ago

go buildではなく go build cmd/cnet/main.goですね

https://qiita.com/yu-croco/items/8f1f8e10246e6f058636

Tomo-9925 commented 3 years ago

プロジェクトのルートからビルドすることを想定していませんでした… これやったら行けるかな…?

Go言語 - 複数ファイルのコンパイル - hakeの日記

Tomo-9925 commented 3 years ago

これでも行けるのか…

~/g/g/T/cnet feature/fix_log_and_errors k992077@k992077_sotsuken
❯ go build cmd/cnet/*.go

~/g/g/T/cnet feature/fix_log_and_errors k992077@k992077_sotsuken
❯ echo $status
0

importに書いたほうが良いですかね…?ファイルわけない方が良かったりします…?ベストな方法はどれでしょうか?

masibw commented 3 years ago

なるほどーー それは知りませんでした、すいません

うーん…それは難しいですね 分けることは悪くないと思うので一旦はそのビルド方法で良い気がします

masibw commented 3 years ago

とりあえず logrusが改行文字をエスケープしてるっぽいことは発見しました https://github.com/sirupsen/logrus/issues/608

ただ、なぜそもそも改行文字とかが出てきてるのかって感じですね

Tomo-9925 commented 3 years ago

チャンネルのポインタについてはそのままで行きます. ポリシーのゼロ値についてはprocの構造体と共有しているため,ややこしくなる上に処理が増えてしまうため,正直変更したくないなーという感じです. gopacket.String()については表示されるのが,Debug時だけということを考えると,fmt.Sprintf()とかを使用して整形する必要があるのかという感じですよね…

Tomo-9925 commented 3 years ago

このlintのエラーってGithub Actionsの設定の方ですかね…?

masibw commented 3 years ago

チャンネルのポインタについてはそのままで行きます.

了解です!

ポリシーのゼロ値についてはprocの構造体と共有しているため,ややこしくなる上に処理が増えてしまうため,正直変更したくないなーという感じです.

あーそれは触りたくないですね...

gopacket.String()については表示されるのが,Debug時だけということを考えると,fmt.Sprintf()とかを使用して整形する必要があるのかという感じですよね…

難しいところですね・・別に直さなくても良さそうな気がしています

masibw commented 3 years ago

このlintのエラーってGithub Actionsの設定の方ですかね…?

なんかそうみたいですね 修正するようのPR出しておきます