Gestire HTTP_STATUS_CODE 409 - Conflict

Dante ha restituito, ad ew2openapi, un http_status_code=409 per l'inserimento di un pick-scnl a causa di un conflitto sul DataBase nell'inserimento di un SCNL.

Analizzando i log per la stazione incriminata MCIV.CHZ.EL.-- (che riporto sotto), penso che il problema sia dovuto al fatto che all'incirca allo stesso istante (2025-03-12 19:44:05.87) il pick-scnl sia stato inserito sia da hew1 che da hew2; l'inserimento di una SCNL avviene mediante SELECT or CREATE.
Quindi:

• hew2 ha fatto la SELECT su scnl, non ha trovato risultato e l'ha inserito (INSERT)
• hew1 ha fatto la SELECT su scnl, non ha trovato risultato e l'ha inserito (INSERT) ma ha ottentuto un duplicate key value

questo ha portato a "buttare" il pick.

Proposta: In caso di 409, ritenterei per un numero finito di volte a distanza di qualche secondo.

hew2:[2025-03-12 19:44:05.878158 - 2joAR] - API-v3.38.0.INFO: {"data":{"ewLogo":{"type":"TYPE_PICK_SCNL","module":"MOD_PICK_EW","installation":"INST_INGV","instance":"hew2","user":"ew","hostname":"af638c27a58b"},"ewMessage":{"pickId":11157682,"network":"EL","station":"MCIV","component":"CHZ","location":"--","firstMotion":"D","pickWeight":2,"timeOfPick":"2025-03-12T19:43:50.950Z","pAmplitude":[3997,3164,1802]}}} [] []

hew1:[2025-03-12 19:44:05.875564 - Ip6ul] - API-v3.38.0.INFO: {"data":{"ewLogo":{"type":"TYPE_PICK_SCNL","module":"MOD_PICK_EW","installation":"INST_INGV","instance":"hew1","user":"ew","hostname":"dae103749e69"},"ewMessage":{"pickId":10929569,"network":"EL","station":"MCIV","component":"CHZ","location":"--","firstMotion":"D","pickWeight":2,"timeOfPick":"2025-03-12T19:43:50.950Z","pAmplitude":[3997,3164,1802]}}} [] []

hew1:[2025-03-12 19:44:05.972074 - Ip6ul] - API-v3.38.0.ERROR: SQLSTATE[23505]: Unique violation: 7 ERROR:  duplicate key value violates unique constraint "idx_110003_sta" DETAIL:  Key (sta, cha, loc, net)=(MCIV, CHZ, --, EL) already exists. (SQL: insert into "scnl" ("net", "sta", "cha", "loc") values (EL, MCIV, CHZ, --) returning "id") {"exception":"[object] (Illuminate\\Database\\QueryException(code: 23505): SQLSTATE[23505]: Unique violation: 7 ERROR:  duplicate key value violates unique constraint \"idx_110003_sta\"\nDETAIL:  Key (sta, cha, loc, net)=(MCIV, CHZ, --, EL) already exists. (SQL: insert into \"scnl\" (\"net\", \"sta\", \"cha\", \"loc\") values (EL, MCIV, CHZ, --) returning \"id\") at /app/vendor/laravel/framework/src/Illuminate/Database/Connection.php:712)\n[previous exception] [object] (PDOException(code: 23505): SQLSTATE[23505]: Unique violation: 7 ERROR:  duplicate key value violates unique constraint \"idx_110003_sta\"\nDETAIL:  Key (sta, cha, loc, net)=(MCIV, CHZ, --, EL) already exists. at /app/vendor/laravel/framework/src/Illuminate/Database/Connection.php:373)"} []

added critical label

assigned to @matteo.quintiliani

Googlando... in merito ad API che ricevono un duplicate key del DB sottostante:

- 409 Conflict: viene utilizzato quando una richiesta non può essere completata a causa di un conflitto con lo
stato attuale della risorsa. Ad esempio il database rifiuta l'inserimento perché la combinazione è già presente 
e violerebbe un vincolo di unicità.

- 428 Precondition Required" sarebbe meno adatto perché si usa quando il server richiede che una 
condizione (come un If-Match o If-None-Match) sia soddisfatta prima di accettare la richiesta. 
Non è  il caso di "duplicate key", perché il problema qui è la violazione del vincolo di unicità, 
non la mancanza di una precondizione.

In base a quanto letto sopra, devo ricontrollare anche gli altri 428; potrebbero non essere corretti.

https://gitlab.rm.ingv.it/earthworm/ew2openapi/-/blob/36b79d9fbe0c490630cf3e9adfb418a5abf6bdf6/src/ew2openapi_handler_quakedb_ws.c#L308

		case 409: // Conflict
		case 428: // Precondition Required
			/* Set handler_status_code to HANDLER_STATUS_ERROR_LIMITED_RETRIES_WITHIN_TIME */
			handler_status_code = HANDLER_STATUS_ERROR_LIMITED_RETRIES_WITHIN_TIME;
			break;

mentioned in commit c6b3fa13

Implement handling of http code 409 (Conflict) with HANDLER_STATUS_ERROR_LIMITED_RETRIES_WITHIN_TIME. Max 60 seconds. Issue #74 (2025/03/14). Commit c6b3fa13

Rilasciata versione 1.19.0

mentioned in commit earthworm-docker-sandbox@7e4c4e67

Hotfix 2.2.5 Update ew2openapi 1.19.0 (f292c846) Issue #74 (2025/03/19). Commit earthworm-docker-sandbox@7e4c4e67

mentioned in commit earthworm-docker-sandbox@c66f604a