Design: Authorization-context-isolated result caching (OBO / session-context)
@croblesm ya está trabajando en esto.
Desde el 24/8/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Problem
Result caching is currently forced off when OBO (user-delegated-auth) or MSSQL set-session-context is enabled:
- OBO: config validation requires
runtime.cache.enabled = false - Session-context:
CanUseCache()returns false so the cache is never used
That is safe but overly conservative. The database identity is already isolated per request, yet operators must choose between per-user authorization and caching (or run dual instances).
Database policies (@claims.*) also cause many read paths to skip the cache even when caching is otherwise enabled.
Proposal
Partition result-cache keys by an authorization-context fingerprint so cached results cannot cross security boundaries:
ResultCacheKey = QueryFingerprint + auth:{Fingerprint} + v:{EntityVersion}
- Fingerprint inputs: client role (
X-MS-API-ROLE), subject (oid/sub), tenant (tid), and either policy-referenced claims (when static analysis is complete) or the full non-volatile principal (session-context / incomplete analysis). - Fail-closed: if the fingerprint cannot be computed, that request skips cache get and set.
- Mutations: bump a per-entity version token so all auth partitions miss after a write (same process).
- Non-isolated entities: keep the existing key shape (no
auth:segment).
Security invariant
Share a cache entry only when every result-affecting security input matches. The previous "disable cache under OBO/session-context" rule is replaced by explicit isolation, not by weakening isolation.
Scope (v1)
- SQL query path (REST + GraphQL via
SqlQueryEngine) - OBO and
set-session-contextare MSSQL features; other engines keep current semantics aside from the shared entity-version segment when an entity name is supplied - Opt-in via existing
runtime.cache.enabled/ entity cache settings (defaults unchanged)
Proposed v1 decisions (for maintainer alignment)
| Open question | Proposed v1 answer |
|---|---|
| Multi-instance / L2 Redis invalidation | Process-local entity version + TTL. Document that cross-replica staleness is bounded by cache TTL. Shared Redis version key is a follow-up when L2 is configured. |
@claims.* static analysis |
Regex extractor at config load for @claims.<name> in database policy strings. Richer policy AST analyzer only if review finds gaps. |
| Observability | Logging only in v1 (e.g. fail-closed at Warning). Metrics for isolated hit/miss / fail-closed as a follow-up. |
Multi-instance wording (proposed for the design contract)
Multi-instance invalidation: Entity version counters are process-local in the initial design. Cross-replica staleness is bounded by cache TTL. A follow-up can move the version token into L2 (Redis) when distributed cache is enabled (
INCRon mutation, include value in the key on read), preserving the same key shape and fail-closed behavior without requiring per-partition eviction.
Validation status (prototype)
- Unit coverage for fingerprint isolation, fail-closed, claim analysis, and config validation
- Live Entra OBO + cache smoke on MSSQL
- Full multi-engine CI and a dedicated TestServer JWT suite remain follow-ups
Feedback on the isolation model and the multi-instance invalidation approach would be especially useful before an implementation PR.
- Lenguaje dominante
- C#
- Estrellas
- 1.5k
- Forks
- 372
- Merge medio
- 9 d 1 h
- PR fusionados (30 d)
- 13
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 Azure/data-api-builder
-
pgsql
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Azure/data-api-builder#3598 ·
-
2.x cli mcp-server
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Azure/data-api-builder#3576 ·
-
2.x health-endpoint
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Azure/data-api-builder#3570 ·
-
2.x telemetry
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Azure/data-api-builder#3564 ·
-
2.x telemetry
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Azure/data-api-builder#3562 ·
Todos los issues de Azure/data-api-builder
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
bug needs response
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Adyen/adyen-dotnet-api-library#1869 ·
-
Status: Needs Triage Type: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
bug effort:S P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
nightscout/nocturne#1602 ·