Repository navigation
api: do not change passwords when their value is "<redacted>" (#6141) (#6287) - #6292
Merged
Merged
Conversation
aler9
force-pushed
the
fix/patch-redacted
branch
from
October 5, 2026 15:30
7c7998c to
c45c677
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6292 +/- ##
=========================================
+ Coverage 0 65.25% +65.25%
=========================================
Files 0 277 +277
Lines 0 24082 +24082
=========================================
+ Hits 0 15714 +15714
- Misses 0 7121 +7121
- Partials 0 1247 +1247 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
aler9
force-pushed
the
fix/patch-redacted
branch
from
October 5, 2026 16:14
c45c677 to
92c0f5f
Compare
…#6287) some users call /config/global/get and pass its content to /config/global/patch. Since the former now redacts passwords and replaces their value with "<redacted>", this string is used as an effective password, breaking authentication. Now password values equal to "<redacted>" are ignored by /config/global/patch, and the original password value is kept unchanged.
aler9
force-pushed
the
fix/patch-redacted
branch
from
October 5, 2026 16:30
92c0f5f to
a6267b5
Compare
Contributor
|
This issue is mentioned in release v1.21.2 🚀 |
xodbs1021
added a commit
to 3K-PokeClip/PokeClip-mono
that referenced
this pull request
Oct 12, 2026
5단계 종결 1회차 리뷰 처분의 「수정」 행(1 · 3 · 6 · 7 · 8)과 추가 처분 9(kty 94차)를 한 커밋으로 고친다. 핀 값(FROM · dev-media image · 버전 계약 상수)과 시험은 바꾸지 않는다. 처분 1(문서 H) — 포크 수정 개수 서술과 포크 걷어내기 조건 - Dockerfile.mtxhook · version_contract_test.go(mediaMTXImage 주석 · forkPinGuide) · README 의 「4건」 서술에서 개수를 지우고 README "이미지 출처에 묶인 전제" 절 표를 단일 정본으로 가리킨다. - 걷어내기 조건 = 그 표의 모든 행이 상류 릴리스에 들어가면(0b 처럼 상류가 다른 수정으로 대체한 것은 bluenviron/mediamtx#6326 이 들어가면). 절차 앞에 U(bluenviron/mediamtx#6306)가 표에 있고, U 없이 공식 이미지로 되돌리면 dev-media 계정별 방송(후속 PR) 뒤 모든 SRT 송출이 경로 잠금값에 막힌다는 것을 적었다. - forkPinGuide 목록에 bluenviron/mediamtx#6292(체리픽 — 상류 v1.21.2 에 있음 · 라인 베이스가 v1.21.1 이라 싣는다)와 U 를 더하고, 「되돌리면 잃는 것」에 U 를 잃는 결과를 더했다. - README 「되돌린 v1.20.1-pokeclip.2 에도 위 4건이 없다」를 「위 표의 수정이 없다」로. 그 태그는 상류 f82bc23 위에 슬레이트 스위치 3커밋과 라인 인프라 3커밋뿐이다(GitHub compare 로 확인). 처분 3(문서 M) — README #217 되돌림 절 앞에 v1.21.1-pokeclip.2 되돌림 다섯 줄: 머지 커밋 revert -m 1 → .1, U 를 쓰는 후속 PR(dev-media 계정별 방송)이 머지돼 있으면 그것을 먼저(머지의 역순 — #217 절과 같은 꼴). 처분 6(문서 L) — README 지금 핀 예시 태그를 v<상류>-pokeclip.<N> 으로. 처분 7(문서 L) — 0b 를 「닫힘(2026-10-11) — 상류는 bluenviron/mediamtx#6326 으로 다르게 고침(릴리스 전)」으로(README 표 · Dockerfile · forkPinGuide). 수정 9 · 10 은 「심사 중」 그대로. 처분 8(문서 L) — 「포크 태그를 올릴 때」 확인 둘을 절차대로 다시 돌려 표적 재확인 기록에 적었다. 두 아치 /mediamtx(docker create · docker cp)의 vcs.revision 과 발행 실행 headSha 가 ff1f75bb, vcs.modified=false, 박힌 mediacommon/v2 v2.9.5 · go-mp4 v1.7.3 = media/go.mod. 두 태그 사이 internal/recorder · internal/formatprocessor · go.mod 변경 0파일이라 픽스처 재채취는 면제다. 원문은 작업 공간 p5_bench/08_fork_checks.txt. 처분 9(kty 94차) — ADR-040 2026-08-30 추기 「두 핀은 같이 올린다」를 2026-10-12 추기로 바꾼다. dev-media(서버) 핀은 서버가 필요로 할 때 서버 기준으로 올리고, 제품 핀(Dockerfile.mtxhook FROM — 팀 로컬)과 같을 필요는 없다. 자리: infra/dev-media/compose.yml 11행 줄 끝 주석 · Dockerfile 머리 · FROM 위 주석 · 걷어내기 절 · forkPinGuide 5번과 걷어내기 1번 · README 네 곳. 걷어내기 절차의 compose 줄은 「서버가 기대는 수정(특히 U)을 기준으로 따로 판단한다」. grep 증거(전수 — 남은 0건) - 개수: git grep -n -E '4건|4 건|네 건|수정 넷|미머지 수정|그중 둘|나머지 둘' -- media/Dockerfile.mtxhook media/internal/mtxhook media/README.md infra docs → 포크 수정 개수 서술 0건. 남은 줄은 README 1259(1.21.1 재검증 기록 — 과거 판, 참) · README 327(무관 「kty 확정 4건」) · docs 의 「34건」 · 「14건」(부분 일치, 무관). - 걷어내기 조건: git grep -n -E '전부 상류 릴리스에|모든 행이 상류 릴리스' -- media infra docs → 「4건이 전부」 0건, 세 자리 모두 「표의 모든 행」. - 0b 상태: git grep -n -E '#6206 \(심사 중\)|6206 · bluenviron/mediamtx#6259' -- media infra docs → 0건. - 옛 두 핀 규칙: git grep -n -E '두 핀|같이 올린|같이 올려|같은 좌표|같은 포크 좌표|같은 공식 좌표|2026-08-30 추기' -- media infra docs → 옛 규칙 서술 0건. 남은 docs/dev-environment.md:425 는 08-30 추기의 「유지 확정」 이라 다른 뜻이다(docs/ 는 이 커밋 범위 밖). 검증: go build · go vet · gofmt -l(빈 출력) · go test -count=1 ./... (17패키지 ok) · 하네스 타입체크 · check-style · git diff --check 통과. pinnedMediaMTXTag 를 작업 트리에서만 틀린 값으로 바꿔 실패 메시지에 새 목록이 나오는 것을 보고 되돌렸다.
8 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
some users call /config/global/get and pass its content to /config/global/patch. Since the former now redacts passwords and replaces their value with "<redacted>", this string is used as an effective password, breaking authentication. Now password values equal to "<redacted>" are ignored by /config/global/patch, and the original password value is kept unchanged.