apache/gravitino

[Improvement] Use of regex on error messages can be fragile in DorisExceptionConverter.java

Aperta

#3027 aperta il 19 apr 2024

Vedi su GitHubGuida contributor
 (11 commenti) (0 reazioni) (1 assegnatario)Java (887 fork)auto 404
good first issueimprovement

Metriche repository

Star
 (3058 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

What would you like to be improved?

DorisExceptionConverter.java uses a DATABASE_ALREADY_EXISTS_PATTERN_STRING regex to check error messages for databases that already exist. There are several issues with this and regex can be fragile expensive or even a possible DOS vector

How should we improve?

If possible improve code to not use regexp for this error condition.

Guida contributor

Direzione di ricerca
Esamina l'uso dell'espressione regolare DATABASE ALREADY EXISTS PATTERN STRING in DorisExceptionConverter.java, identifica il formato del messaggio di errore del driver del database e sostituisci la corrispondenza regex con un controllo diretto della stringa o una gestione strutturata delle eccezioni.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Javaregex
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60