api: commit-pack refuses permission-style file modes that upstream normalizes

オープン
#806 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
49/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
go
領域
api

調査の方向性

Start with internal/api/handlers_write_commit.go:506 and inspect wcValidModes and the commit-pack handler. Trace diff-commit and every other route that accepts FileOperation.mode to compare their validation paths. Normalize the four documented forms before validation, preserve current behavior for other modes, and confirm the affected requests are accepted while other mode responses stay unchanged.

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

説明

Problem

Spec 5d8e1801 says commit-pack stores 644 and 100664 as 100644, and 755 and 100775 as 100755. We answer 400 for all four (wcValidModes, internal/api/handlers_write_commit.go:506).

Desk decision

Normalize exactly those four forms before validation. Every other mode keeps its current answer. Check diff-commit and every other route that takes FileOperation.mode.

Severity

Medium: valid upstream requests fail.

Upstream audit 2026-11-12, evidence 2026-11-12-1159UTC. Docs spec 5d8e1801 (ETag W/"8GUlAlKU8eko7Rr2", Last-Modified 2026-11-10 22:29:59 GMT). The cluster still serves ee8092bf, so this is documented drift, not a proven deployment (#234 rule: adopt what the docs publish). SDK 0.1.1 (Nov 10) and 0.1.2 (Nov 11) shipped on npm, PyPI and Go; their route list and token minting are unchanged from 0.1.0.


Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: medium

主要言語
Go
スター
0
フォーク
0
平均マージ
6時間 16分
マージ済み PR(30日)
243

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

coder/openstorage のほかの issue

coder/openstorage の issue をすべて見る

似ている issue

Go の issue をもっと見る

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

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