nodeSolidServer / nodeSolidServer/node-solid-server

Don't rely on existence of /profile/card to block sign-up.

Đang mở
#1,368 1 bình luận 0 reaction 1 người được giao Xem trên GitHub

@jaxoncreed đang làm issue này rồi.

Từ ngày 10/12/2019.

Ngôn ngữ chính
JavaScript
Star
1.8k
Fork
308
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

In single-user mode, you can register once.
If you try to register a second time, https://github.com/solid/node-solid-server/blob/master/lib/models/account-manager.js#L111 will notice that ./data/profile/card$.ttl already exists, and respond with a 400.

However, if you manually delete that file, then that means that if someone goes to the Register page, they can take over control of your pod.

For multi-user mode, this would presumably work too, but then you need to "guess" the username for which the profile doc is missing, so it's less likely to occur.

We could change the check for user existence, for instance, (also) check if an oidc user exists, or if a data folder exists at all.

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

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

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Đánh giá

Issue này chưa được đánh giá.

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.