Skip to content

Conversation

@sitya
Copy link

@sitya sitya commented Apr 19, 2016

AttributeLimit allows to release only that attributes to the SP, which are listed in SP's metadata. In the most federation metadata is generated by a central tool of the national federation, so there could be attributes which are not specified in the federation (so it could not be listed in SP's requiredAttributes), but the SP would need it.

With this PullRequest we implemented this feature. It adds the functionality of specifying bilateral attribute relations in addition to the attribute rules defined in the peer's metadata. You can both specify SP entityIDs where you send some additional attributes (bilateralSPs) and special attributes to send to some SPs (bilateralAttributes).

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 15.821% when pulling 17d3cee on NIIF:master into 8de8f35 on simplesamlphp:master.

@jaimeperez
Copy link
Member

Hi @sitya! Thanks a lot for the PR!

After a very, very quick glance at it, it looks great. I'll take a deeper look at it as soon as I can and hopefully merge it 😉

@tvdijen tvdijen force-pushed the master branch 7 times, most recently from 3b5f5ba to 96357ee Compare July 19, 2023 12:37
@tvdijen tvdijen force-pushed the master branch 5 times, most recently from 7587851 to d523b31 Compare August 2, 2023 11:58
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from 8c90121 to d534e3b Compare September 5, 2023 08:09
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from bc1c5c8 to d0a5974 Compare October 17, 2023 21:17
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from ccb9b02 to 120a100 Compare December 1, 2023 14:34
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 6004a77 to 58bf8db Compare May 4, 2024 23:45
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 5c9fb2c to 0970efc Compare May 27, 2024 12:27
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from c27831c to 71e49f4 Compare June 19, 2024 17:03
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from c06a17a to a52c98d Compare August 20, 2025 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants