fix(platform_measurement): reject non-string or empty runtime.measurement across Python and TypeScript
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #633 di @drk-ram — integrata
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 30/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- python, typescript
- Ambito
- backend-api-design
Direzione di ricerca
Read src/agentrust_trace/platform_measurement.py and trust-record-verify/src/platform.ts, then find the tests for check_platform_measurement and checkPlatformMeasurement. Check how each function handles absent, empty, and non-string runtime.measurement values, and verify the returned outcome and cause against the proposed behavior. Pull request #633 is already open for this issue.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
Direct calls to check_platform_measurement (src/agentrust_trace/platform_measurement.py) and checkPlatformMeasurement (trust-record-verify/src/platform.ts) do not require runtime.measurement to be a non-empty string:
- Non-string or empty
runtime.measurementaccepted incheck_platform_measurement/checkPlatformMeasurement:- Both implementations check only
runtime.get("measurement") is None(ormeasurement === undefined || measurement === nullin TypeScript). - If
record={"runtime": {"measurement": 123}},record={"runtime": {"measurement": ""}}, orrecord={"runtime": {"measurement": True}}is passed directly and the caller's appraiser echoes{"measurement": runtime["measurement"], "layers": {"pcr:0": "established"}}, both functions returnoutcome="appraised"with a non-string or emptyevidence["measurement"], violating the documented contract thatmeasurementis a non-empty string ("measurement, the string it appraised") andevidenceis JSON-serializable.
- Both implementations check only
Proposed Fix
- In
check_platform_measurement(Python) andcheckPlatformMeasurement(TypeScript), requireisinstance(measurement, str) and bool(measurement)(in TS:typeof measurement === "string" && measurement !== ""), returningnot_attemptedwithcause="field_absent"whenruntime.measurementis absent, empty, or not a string.
- Lingua principale
- Python
- Stelle
- 37
- Fork
- 33
- Merge medio
- 1g 13h
- PR unite (30g)
- 120
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di agentrust-io/trace-spec
-
needs-review:MEDIUM
Difficoltà 4/5 3-5 giorni Idoneità per principianti 1/100
agentrust-io/trace-spec#640 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
prototype(verifier_token,execution_appraisal): enforce canonical 32-byte nonces, validate RelyingParty.challenge/verify_proof inputs, and reject non-finite JWT JSONForse già presa @drk-ram l’ha presa 4 giorni fa. Apertaneeds-review:MEDIUM
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
agentrust-io/trace-spec#629 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
fix(provenance,validate): remove str() coercion on digest/hash fields in provenance and enforce strict JSON in schema loadersForse già presa @drk-ram l’ha presa 4 giorni fa. Apertaneeds-review:MEDIUM
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
agentrust-io/trace-spec#627 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
conformance(runner): reject boolean/float level, invalid fmt, and malformed caller options at run() entryForse già presa @drk-ram l’ha presa 4 giorni fa. Apertaneeds-review:MEDIUM
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
agentrust-io/trace-spec#626 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
conformance(inclusion): enforce registry-anchor-v1 surrogate/key/cycle rules and validate leaf_index/entry fieldsForse già presa @drk-ram l’ha presa 4 giorni fa. Apertaneeds-review:MEDIUM
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
agentrust-io/trace-spec#625 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di agentrust-io/trace-spec
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Graphify-Labs/graphify#4860 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
:bug: Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
scikit-image/scikit-image#8377 ·
I maintainer di solito rispondono entro 1 giorno
-
Ingredient import validation accepts negative nutrient values and NaNForse già presa @LukeSekai l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
wger-project/wger#2564 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
apache/ponymail-foal#351 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
I maintainer di solito rispondono entro 4 giorni