Problem/Motivation

Add a new configuration key to allow repeat attribute name in order to support Attribute Statements with Attribute elements with the same name.

Proposed resolution

Having a new configuration key security_allow_repeat_attribute_name disabled by default.

When checked the toolkit will not raised an error on duplicated Statement Element containing duplicate attribute elements.

CommentFileSizeAuthor
#2 3252457-02.patch3.08 KBwengerk

Comments

wengerk created an issue. See original summary.

wengerk’s picture

wengerk
he/him
Primary language French
Location Lausanne
commented
StatusFileSize
new 3252457-02.patch3.08 KB

Here is a first suggestion patch to allow this new feature

wengerk’s picture

wengerk
he/him
Primary language French
Location Lausanne
commented
Status: Active » Needs review
wengerk’s picture

wengerk
he/him
Primary language French
Location Lausanne
commented
Title: add config to allow repeat attribute name » Add config to allow repeat attribute name

  • roderik committed 798b49a on 8.x-3.x authored by wengerk 
    Issue #3252457 by wengerk: Add config to allow repeat attribute name
roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented

Thank you; committed. As all extra library options, this is a good addition.

You may need to manually reconcile what I committed; I'm not committing the "?? TRUE" part for SamlService::reformatConfig(), to stay close to the library's defaults.

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
Status: Needs review » Fixed

  • roderik committed 798b49a on 4.x authored by wengerk 
    Issue #3252457 by wengerk: Add config to allow repeat attribute name

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.