simplesamlphp / simplesamlphp/simplesamlphp-module-ldap
Have ldap:AttributeAddUsersGroups support ldap:LdapMulti
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- PHP
- Star
- 5
- Fork
- 14
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
Hi, I noted that in the docs for ldap:AttributeAddUsersGroups, it is clearly stated:
Note: ldap:LdapMulti is not supported as the SimpleSAMLphp framework does not pass any information about which LDAP source the user selected.
And I understand that if you do have multiple LDAP instances, you can place your ldap:AttributeAddUsersGroups configuration in the service provider metadata and in that way control which LDAP instance is queried for groups when a user attempts to access a particular service.
But I do wonder if there is any appetite for adding support for ldap:LdapMulti? So that with ldap:AttributeAddUsersGroups configuration in the IdP metadata, groups will be queried from the authentication source (organisation) the user selected?
I'd be very happy to work on this if anyone feels it would be a useful feature to add.
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 với docs/ldap.md và phần ldap:AttributeAddUsersGroups để hiểu giới hạn đã được ghi lại. Sau đó, truy vết cách siêu dữ liệu IdP và nguồn xác thực được chọn được biểu diễn, vì issue xác định rằng đang thiếu thông tin về framework nhưng không nêu điểm bắt đầu triển khai hoặc bài kiểm thử. Để hoàn thành, cần có một thiết kế đã được thống nhất, hỗ trợ ldap:LdapMulti và cập nhật tài liệu.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- php
- Lĩnh vực
- authentication
- Loại issue
- Tính năng
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 30/100