Skip to content

Commit 8d9d928

Browse files
author
Ana Martinez
committed
Memoize name_id_node
1 parent 9478e8b commit 8d9d928

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

lib/samlr/assertion.rb

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,11 @@ def attributes
4848
end
4949

5050
def name_id
51-
@name_id ||= assertion.at("./saml:Subject/saml:NameID", NS_MAP).text
51+
@name_id ||= name_id_node.text
5252
end
5353

5454
def name_id_options
55-
@name_id_options ||= Hash[assertion.at("./saml:Subject/saml:NameID", NS_MAP).attributes.map{|k,v| [k, v.value]}]
55+
@name_id_options ||= Hash[name_id_node.attributes.map{|k,v| [k, v.value]}]
5656
end
5757

5858
def conditions
@@ -61,6 +61,10 @@ def conditions
6161

6262
private
6363

64+
def name_id_node
65+
@name_id_node ||= assertion.at("./saml:Subject/saml:NameID", NS_MAP)
66+
end
67+
6468
def assertion
6569
@assertion ||= document.at(location, NS_MAP)
6670
end

0 commit comments

Comments
 (0)