CenterForDigitalHumanities / CenterForDigitalHumanities/rerum_server_nodejs

If-Overwritten-Version header not enforced when accessed via TinyNode

Aperta
#230 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
3
Fork
6
Merge medio
2g 47m
PR unite (30g)
4

Descrizione

## Summary
The `If-Overwritten-Version` header for optimistic locking is documented in the RERUM API but does not appear to be enforced when `/overwrite` requests come through TinyNode (tiny.rerum.io).

## Background
According to the [RERUM API documentation](https://store.rerum.io/v1/API.html#overwrite) and the server code in `controllers/overwrite.js`, the `If-Overwritten-Version` header should be checked against `__rerum.isOverwritten` to detect concurrent edit conflicts.

The server code shows this logic:
```javascript
const expectedVersion = req.get('If-Overwritten-Version') ?? req.body.__rerum?.isOverwritten
const currentVersionTS = originalObject.__rerum?.isOverwritten ?? ""
if (expectedVersion !== undefined && expectedVersion !== currentVersionTS) {
// Should return 409 Conflict
}
```

## The Problem
When accessing RERUM through TinyNode (`tiny.rerum.io/overwrite`), the `If-Overwritten-Version` header is sent correctly but the version check is not enforced - requests succeed even with version mismatches.

## Steps to Reproduce
1. Load an object via `store.rerum.io/v1/id/{id}` - note `__rerum.isOverwritten` (e.g., `2025-11-30T20:31:57.853`)
2. Send `/overwrite` to `tiny.rerum.io` - this updates `isOverwritten` to new value
3. Send another `/overwrite` to `tiny.rerum.io` with header `If-Overwritten-Version: 2025-11-30T20:31:57.853` (old value)
4. **Expected:** 409 Conflict
5. **Actual:** 200 OK, silent overwrite

## Questions
1. Does TinyNode pass through the `If-Overwritten-Version` header to RERUM?
2. Is the optimistic locking check implemented in TinyNode or expected to be handled by RERUM?
3. Is there a configuration needed to enable this feature?

## Related
- TinyNode issue: https://github.com/CenterForDigitalHumanities/TinyNode/issues/106
- RERUM API docs: https://store.rerum.io/v1/API.html#overwrite

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia da controllers/overwrite.js e segui il percorso della richiesta /overwrite, quindi confrontalo con la gestione di /overwrite di TinyNode e con la issue 106 correlata di TinyNode. Verifica se If-Overwritten-Version raggiunge il server RERUM e se valori non corrispondenti producono 409 Conflict invece di 200 OK. Il lavoro è completato quando il comportamento documentato di optimistic locking viene applicato oppure viene identificata chiaramente la lacuna di integrazione/configurazione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
api, backend
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.