doctrine/sql-formatter

Cannot render SQL queries containing binary strings

Open

#91 aperta il 21 lug 2023

Vedi su GitHub
 (4 commenti) (3 reazioni) (0 assegnatari)HTML (17 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (1559 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

At the moment, even though valid, these kind of queries would crash doctrine-bundle + twig

$connection->executeQuery("SELECT {$connection->quote((new \Symfony\Component\Uid\Ulid())->toBinary())}");

We are running all of these queries through this formatter, so I think formatter should do something about these, eg. detect non-utf8 parts and replace them with something.

Original issue was https://github.com/doctrine/DoctrineBundle/issues/1315, I'm moving it here

Guida contributor

Direzione di ricerca
Esamina il codice sorgente del formattatore SQL, in particolare le classi tokenizer e formatter, per capire come le query SQL vengono analizzate e generate in output. Cerca come i caratteri non UTF8 sono attualmente gestiti (probabilmente non lo sono). La correzione potrebbe comportare il rilevamento di segmenti binari all'interno di stringhe tra virgolette e la loro sostituzione o escape. Rivedi i test esistenti nel repository per vedere se ce ne sono che coprono i dati binari, e considera di aggiungere un test con una query ULID to binary simile all'esempio del problema. Controlla il problema DoctrineBundle referenziato per ulteriore contesto sul crash.
Tech stack
php
Dominio
database
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
PHPSQLbinary strings
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60