Allow optional prefix for AuthNRequest ID attribute#630
Allow optional prefix for AuthNRequest ID attribute#630gowda wants to merge 2 commits intoSAML-Toolkits:masterfrom
Conversation
|
It makes more sense to add the prefix to the uuid method ot the Utils, rather than in only the request constructor. You can add a constant to the Utils class by default "_" (the current prefix) so then later the user will be able to override it dinamycally |
I have update |
|
I dislike the need of changing code in all the places where uuid is used with the new parameter. Can you instead follow my suggestion and just add a param to the Util class, by default "_" and then change the uuid in order to use it. developers that want the new behavior only will need to change the attribute from the Util class and that all. |
|
prefix support gonna be included at #633 |
Support ability to change ID prefix. See #630
IDattribute is returned asInResponseToattribute of the response by servers (as mentioned by Azure documentation).