fill: a sensitive value can still come back in an altered form (iOS read-back)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- swift, typescript
Línea de trabajo
Start with the TEXT_ENTRY_MISMATCH message in RunnerTests+TypeExecution.swift around line 111, which quotes the observed read-back value, and with FillUnconfirmedVerification in packages/contracts/src/fill-evidence.ts, which carries before and after. Check how #3351 redacts the expected and requested values for sensitive fills, then apply the same treatment to observed, before and after. Done means a sensitive fill that fails or settles on a different value returns no read-back text in the error or the response, while the error code and verification status remain. A device run is not required to verify the change, but the design of the placeholder should be agreed first.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Follow-up from #3351, as suggested in review.
#3351 keeps the exact text of a sensitive fill (--text-stdin, --record-as) out of responses and errors. Two places can still carry a changed form of that text, which no literal match catches:
- Error: the iOS runner's
TEXT_ENTRY_MISMATCHmessage quotes both sides:text entry read-back did not match: expected "…", observed "…"(RunnerTests+TypeExecution.swift:111).expectedis the exact value and is now redacted.observedis what the field read back. Per thefillhelp this error covers dropped characters, so it can be the value with a character or two missing. - Success: a fill that settles on a different value returns
verification: "unconfirmed"withrequested,beforeandafter(FillUnconfirmedVerificationinpackages/contracts/src/fill-evidence.ts).requestedis the exact value, butaftercan be a formatted form of it, like the help's"00 062 91 77"for"000629177".
I found these by reading the code and have not reproduced them on a device, since that needs a field that alters its input.
A possible direction: for a sensitive fill, drop the read-back values instead of matching them. For example, keep the error code with a generic message, and replace before/after with the placeholder. Related: #3261 (the same value in native XCTest logs).
- Lenguaje dominante
- TypeScript
- Estrellas
- 4.9k
- Forks
- 328
- Merge medio
- 12 h 18 min
- PR fusionados (30 d)
- 561
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de callstack/agent-device
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
callstack/agent-device#3407 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
callstack/agent-device#3353 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
callstack/agent-device#1869 ·
Los mantenedores suelen responder en 1 día
-
needs-triage
Dificultad 3/5 1-2 días Aptitud para principiantes 76/100
callstack/agent-device#3453 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
callstack/agent-device#3451 ·
Los mantenedores suelen responder en 1 día
Todos los issues de callstack/agent-device
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
feat(icons): add [solidping]Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
homarr-labs/dashboard-icons#2939 ·
Los mantenedores suelen responder en 1 día
-
Agent/category reasoning variants are still checked against the stale model cache in chat.paramsAbiertoopencode
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
code-yeongyu/oh-my-openagent#10003 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
Actions on ToastsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
nextcloud-libraries/nextcloud-dialogs#2669 ·
Los mantenedores suelen responder en 1 día