SolidOS / SolidOS/solid-ui

Code does not check for session = null when obtaining webId

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

Chưa có ai nhận issue này.

authentication input-needed
Ngôn ngữ chính
TypeScript
Star
153
Fork
46
Merge trung bình
1 ngày 8 giờ
Pull request đã merge (30 ngày)
7

Mô tả

How to reproduce

  1. Logged out and cleared localStorage
  2. Attempted to login to https://melvin.solid.live/public/RWWCrew/
  3. Click on custom domain, https://melvincarvalho.com/#me

Observed behaviour

Uncaught (in promise) TypeError: Cannot read property 'webId' of null
    at signin.js:732
    at <anonymous>

in

  signInPopUpButton.addEventListener('click', function () {
    var offline = offlineTestID();
    if (offline) return setUserCallback(offline.uri);
    return solidAuthClient.popupLogin({ popupUri: $SOLID_GLOBAL_config.popupUri }).then(function (session) {
      var webIdURI = session.webId;

Expected Behaviour

What I would expect is to redirect me to my oidc provider, let me log in, and then give me a bearer token

Possible fix

https://github.com/solid/solid-ui/blob/master/src/signin.js#L739 should use webIdFromSession

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

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

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.

Hướng nghiên cứu

Bắt đầu tại src/signin.js, xung quanh handler signInPopUpButton gần dòng 739, sau đó so sánh cách xử lý session của nó với webIdFromSession gần dòng 783. Tái hiện luồng đăng nhập custom-domain khi đã đăng xuất và xác minh rằng session null không còn gây ra TypeError webId đã được báo cáo, đồng thời luồng đăng nhập provider và bearer-token dự kiến có thể tiếp tục.

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
authentication, frontend
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
35/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.