Permalink
Browse files

[project @ Consumer: require that op_endpoint be signed in id_res res…

…ponses]

Ignore-this: a0fbd71a105194bac2624f7cff8a3e7a
  • Loading branch information...
1 parent d51e7c9 commit da588c1c787a1c79af5492df08632c2ebeba1ae5 tailor committed Apr 20, 2009
Showing with 3 additions and 2 deletions.
  1. +1 −1 lib/openid/consumer/idres.rb
  2. +2 −1 test/test_idres.rb
@@ -114,7 +114,7 @@ def check_for_fields
when OPENID2_NS
require_fields = basic_fields + ['op_endpoint']
require_sigs = basic_sig_fields +
- ['response_nonce', 'claimed_id', 'assoc_handle',]
+ ['response_nonce', 'claimed_id', 'assoc_handle', 'op_endpoint']
when OPENID1_NS
require_fields = basic_fields + ['identity']
require_sigs = basic_sig_fields
View
@@ -27,7 +27,8 @@ class CheckForFieldsTest < Test::Unit::TestCase
OPENID1_SIGNED = ['return_to', 'identity']
OPENID2_SIGNED =
- OPENID1_SIGNED + ['response_nonce', 'claimed_id', 'assoc_handle']
+ OPENID1_SIGNED + ['response_nonce', 'claimed_id', 'assoc_handle',
+ 'op_endpoint']
def mkMsg(ns, fields, signed_fields)
msg = Message.new(ns)

0 comments on commit da588c1

Please sign in to comment.