Skip to content

Add attributemap claim2name.php to map MS .NET Framework Claim Types to Names#805

Open
gfaust-qb wants to merge 1 commit intosimplesamlphp:masterfrom
gfaust-qb:patch-3
Open

Add attributemap claim2name.php to map MS .NET Framework Claim Types to Names#805
gfaust-qb wants to merge 1 commit intosimplesamlphp:masterfrom
gfaust-qb:patch-3

Conversation

@gfaust-qb
Copy link
Copy Markdown

@gfaust-qb gfaust-qb commented Mar 2, 2018

Hi,

second, clean try ;)

I needed a mapping for claim types to names several times and started to create my own attributemap.

I added an attributemap that maps Microsoft .NET Framework (standard) ClaimTypes to Names.
At the moment Names are used as used by Microsoft in their description.
@see https://msdn.microsoft.com/en-us/library/hh159803.aspx
I recognized that these Names do not map to the names in name2claim.php or other attributemaps - but that there are inconsistencies in other mappings also.
Do you have a suggestion what Names I should use to make a valuable contribution to this project?

Guido

@gfaust-qb
Copy link
Copy Markdown
Author

Hi,

I don't know why codeclimate fails, log shows nothing depending to this file.
Does it fail because the file is new?

Guido

@tvdijen
Copy link
Copy Markdown
Member

tvdijen commented Mar 6, 2018

Thanks @gfaust-qb ! Very useful!!
You can safely ignore the codeclimate-failure.

Copy link
Copy Markdown
Member

@tvdijen tvdijen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To get to your question.. I think you should use the names of the AD attributes..
So you would use c instead of country for example

@gfaust-qb
Copy link
Copy Markdown
Author

Thank you!

I opened an Issue (#807) to discuss this and to offer my help to generate an mapping overview.

Guido

Maps Microsoft .NET Framework ClaimTypes to Names
Names in this version are used as described by Microsoft.
@see https://msdn.microsoft.com/en-us/library/hh159803.aspx
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 6, 2020

Codecov Report

Merging #805 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #805   +/-   ##
=========================================
  Coverage     38.36%   38.36%           
  Complexity     3385     3385           
=========================================
  Files           128      128           
  Lines          9633     9633           
=========================================
  Hits           3696     3696           
  Misses         5937     5937           

@tvdijen tvdijen force-pushed the master branch 3 times, most recently from 1c686ab to eb20457 Compare August 17, 2020 20:43
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 08ebb9c to 64fca25 Compare July 2, 2021 14:12
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 7a53fc8 to d73ae47 Compare September 26, 2021 13:03
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from e5c0e21 to d5616df Compare January 9, 2022 11:00
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from 2e6ab04 to 32f9acc Compare November 23, 2022 18:24
@tvdijen tvdijen force-pushed the master branch 6 times, most recently from 7e3ea19 to 2523634 Compare January 5, 2023 16:31
@tvdijen tvdijen force-pushed the master branch 7 times, most recently from 7b173cf to 3326beb Compare March 20, 2023 22:59
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 8ac729b to a16cf6e Compare April 25, 2023 08:33
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from fc454de to 7ac76ae Compare May 3, 2023 08:31
@tvdijen tvdijen force-pushed the master branch 6 times, most recently from 29f7b69 to 1a911ce Compare May 12, 2023 16:07
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from c7c8357 to fdbe001 Compare June 12, 2023 14:28
@tvdijen tvdijen force-pushed the master branch 8 times, most recently from 3b5f5ba to 96357ee Compare July 19, 2023 12:37
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from d7f25d2 to 41d9254 Compare July 30, 2023 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants