hackmdio / hackmdio/codimd

Revisions should not url_encode or at least url_encode(diff("content"))

Aperta
#522 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
database/sequelize discussion
Lingua principale
JavaScript
Stelle
10.1k
Fork
1.1k
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

In #521 it looks like the `Revisions` table does something like this:

diff(url_encode("content"))

This makes little sense, as the diffing will always yield all of the document. And those are stored in `lastContent` and `content` anyway.

This should be reversed to be

url_encode(diff("content"))

I'm not even entirely sure why the url-encode is done in the first place, but it appears to be there.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.