Code does not check for session = null when obtaining webId
Chưa có ai nhận issue này.
- 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
- Logged out and cleared localStorage
- Attempted to login to https://melvin.solid.live/public/RWWCrew/
- 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
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- 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