simplesamlphp / simplesamlphp/simplesamlphp-module-oidc

User identifier attribute option as single value is inadequate with heterogenous IdPs scenarios

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

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

Từ ngày 3/6/2025.

enhancement prepared
Ngôn ngữ chính
PHP
Star
50
Fork
28
Merge trung bình
1 phút
Pull request đã merge (30 ngày)
2

Mô tả

Config option for designating user identifier https://github.com/simplesamlphp/simplesamlphp-module-oidc/blob/0080cf2ca10a6c00a80835a9e83638245ce9ea82/config/module_oidc.php.dist#L79 is a single value. This means that it is expected that this attribute is always available in the set of user attributes after user authentication.

In scenarios with multiple different IdPs, like for example in eduGAIN inter-federation, not every IdP will be able (or willing) to provide designated identifier. For this reason, this config option (and all relevant code that uses it) should be refactored in a way to be able to accept multiple prioritized values (array of prioritized values), or be left as single value.

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.

Đá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.