some problem about gzip writer not use sync pool in `middleware/compress.go`
まだ誰も着手していません。
評価
調査の方向性
middleware/compress.go のコミット履歴から始め、削除された gzipPool コードと現在の実装を比較してください。プールが削除された理由を理解するため、リンクされている labstack/armor の issue 13 を読んでください。理由を文書化し、変更が正当化される場合は、gzip writer に期待される動作を定義して関連するテストで検証できれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Looking at the git commit history, I found that the sync pool of the gzip writer was removed. The reason is to fix the issue here: https://github.com/labstack/armor/issues/13. But I still don't understand the reason for this deletion. I guess tha Is it because there is no way to return an error when using sync pool?
the removed code:
func gzipPool(config GzipConfig) sync.Pool {
return sync.Pool{
New: func() interface{} {
// can not return error here
w, _ := gzip.NewWriterLevel(ioutil.Discard, config.Level)
return w
},
}
}
- 主要言語
- Go
- スター
- 1.7k
- フォーク
- 64
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
labstack/armor のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 10/100
-
難易度 1/5 1〜3時間 初心者へのやさしさ 40/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
似ている issue
-
bug github_actions
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
registrystack/registry-stack#1393 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
JakeChampion/lang#10213 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
oasisprotocol/oasis-sdk#2523 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100