apache/gravitino

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

Ouverte

#3 027 ouverte le 19 avr. 2024

Voir sur GitHubGuide contributeur
 (11 commentaires) (0 réaction) (1 personne assignée)Java (887 forks)auto 404
good first issueimprovement

Métriques du dépôt

Stars
 (3 058 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Examinez l'utilisation de l'expression régulière DATABASE ALREADY EXISTS PATTERN STRING dans DorisExceptionConverter.java, identifiez le format du message d'erreur du pilote de base de données et remplacez la correspondance regex par une vérification directe de chaîne ou une gestion structurée des exceptions.
Stack technique
java
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Javaregex
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60