hackmdio / hackmdio/codimd

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

オープン
#522 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
database/sequelize discussion
主要言語
JavaScript
スター
10.1k
フォーク
1.1k
PR マージ指標
30日以内にマージされた PR はありません

説明

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.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Revisions テーブルの実装と、コンテンツに url_encode および diff を適用する #521 関連のコードを特定してください。現在なぜ diff の前にエンコードが行われているのかを確認し、その後、コンテンツに対して diff を行うと焦点の絞られたリビジョンが生成され、URL エンコードが後から適用されることを検証してください。結果を確認するために、関連するテストやリポジトリで利用可能なリビジョンデータがあれば使用してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
databases
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。