Test for trialing slashes in value for 'issuer' from '/.well-known/openid-configuration' responses
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
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- authentication
Hướng nghiên cứu
Bắt đầu bằng cách xác định đoạn mã xây dựng URL /.well-known/openid-configuration từ issuer được trả về bởi phản hồi cấu hình OpenID. Bổ sung coverage cho các giá trị issuer có và không có dấu gạch chéo ở cuối, đồng thời xác nhận rằng cả hai đều tạo ra một URL hợp lệ với chỉ một dấu gạch chéo, thay vì một phản hồi JSON không hợp lệ.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Basically our ESS Brokers are advertising themselves (in their /.well-known/openid-configuration responses) as issuers with a trailing slash, e.g. for https://broker.pod.inrupt.com/.well-known/openid-configuration:
{
"introspection_endpoint": "https://broker.pod.inrupt.com/introspect",
"scopes_supported": [
"openid",
"offline_access",
"webid"
],
"issuer": "https://broker.pod.inrupt.com/",
:
Note the trailing slash on the issuer URL (which is technically (slightly!) more correct than having the URL without the trailing slash).
So it seems NSS might be blindly taking that issuer value, and appending the string literal /.well-known/openid-configuration to it, without first checking if the issuer value has a trailing slash or not. If the URL constructed has two slashes, then the server responds with it’s standard HTML ‘Page Not Found’ response, which results in the ‘invalid json response…’ error.
If this is correct, then the fix in NSS should be simply checking for a trailing slash before appending /.well-known/openid-configuration.
- 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
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 nodeSolidServer/node-solid-server
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
nodeSolidServer/node-solid-server#1848 · 2 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
nodeSolidServer/node-solid-server#1841 · 2 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
nodeSolidServer/node-solid-server#1147 ·
-
remove bootstrap dependency Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
nodeSolidServer/node-solid-server#1867 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 45/100
nodeSolidServer/node-solid-server#1853 ·
Tất cả issue của nodeSolidServer/node-solid-server
Issue tương tự
-
ai-observability bug team/ai-observability
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
AllTheMods/ATM-10-L#19 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
SignalK/n2k-signalk#345 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100
JuliaPluto/PlutoPlotly.jl#72 ·