hackmdio / hackmdio/codimd

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

Đang mở
#522 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
database/sequelize discussion
Ngôn ngữ chính
JavaScript
Star
10.1k
Fork
1.1k
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Xác định phần triển khai của bảng Revisions và mã liên quan đến #521 áp dụng url_encode và diff cho nội dung. Kiểm tra lý do việc mã hóa hiện đang diễn ra trước khi thực hiện diff, sau đó xác minh rằng việc thực hiện diff trên nội dung tạo ra một revision tập trung và URL encoding được áp dụng sau đó; sử dụng mọi bài kiểm thử liên quan hoặc dữ liệu revision có sẵn trong repository để xác nhận kết quả.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
javascript
Lĩnh vực
databases
Loại issue
Lỗi
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
42/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.