Math.random() is not cryptographically secure
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript, nodejs
- Lĩnh vực
- authentication, security
Hướng nghiên cứu
Bắt đầu trong lib/utils.js tại dòng 61, nơi Math.random() được dùng để tạo UIDs. Đọc tài liệu Node.js crypto.randomInt được liên kết và kiểm tra cách các UIDs đó được sử dụng trong toàn bộ thư viện SAML. Hoàn tất khi việc tạo UID không còn phụ thuộc vào Math.random() và vẫn giữ nguyên hành vi về tính duy nhất của thư viện.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
https://github.com/auth0/node-saml/blob/7ee79849d6b0a0935e42ad456ae8cb92d2b3bb93/lib/utils.js#L61
This function is used to generate unique UIDs throughout the SAML library. Given that SAML deals with authentication, it seems like poor practice to not generate random UIDs in a cryptographically secure manner.
Reference: https://nodejs.org/api/crypto.html#crypto_crypto_randomint_min_max_callback
Guidance: https://gist.github.com/joepie91/7105003c3b26e65efcea63f3db82dfba
- Ngôn ngữ chính
- JavaScript
- Star
- 73
- Fork
- 89
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
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.
Issue khác của auth0/node-saml
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 10/100
-
Incompatibility with @xmldom/xmldom 0.9.x: DOMParser.parseFromString throws on undefined mimeTypeCó thể đã có người làm @esetnik đã nhận 156 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
Tất cả issue của auth0/node-saml
Issue tương tự
-
bug MVP
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
saayam-for-all/webapp#1911 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Focus] Accessibility (a11y) [Package] Grid [Type] Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Cancelled query leaves stale row index; next query on the connection returns leading empty slotsCó thể đã có người làm @pislm đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 7 ngày
-
Harden bayesian-waves-bg.js: add prefers-reduced-motion, pointer-events guard, and hidden-tab pauseĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
njitacm/njit-acm-website#109 ·