Age | Commit message (Collapse) | Author | Files | Lines |
|
- added missing 'return' statement to the forward_iterator_proxy operator=()
- made temporary non-static for thread safety in url_decode()
Thanks to Masato Kataoka of orenosv project
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Thanks to Brian Muller for spotting it!
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Thanks to Göran Löwkrantz for pointing both to the problem and possible
solution.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Thanks to Oden Eriksson of Mandriva for report and patch
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Thanks to compiler for spotting it.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
One of the numerous opportunities to express my gratitude by means of commit
message to Joseph Smarr of Plaxo for spotting it. Thanks, Joseph!
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
|
|
work around some thoughtful and considerate RPs that take the pain of sending
the trust_root parameter, but do not go into as much trouble as setting it to
anything but an empty string.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
since oauth fieldsets aren't really messages
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
as per rfc 3986
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
* moved iname leader characters and whitespace characters strings to
opkele::data namespace
* added opkele::util::normalize_identifier() function
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
it foolishly failed in presence of ns field and absence of namespace in
question.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
* added opend_endpoint_output_iterator to types.h
* changed endpoint_discovery_iterator to the aforementioned iterator typedef
* added discover() virtual to prequeue_RP and made use thereof.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
* allocate stateful handles when processing associate request
* use the expiration from allocated handle, not empty shared_ptr
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
and doxygenated basic_OP a bit.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
that would just pollute the database
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
* changed {checkid,id_res}_hook to {rp,op}_{checkid,id_res}_hook
* deprecated older hooks, although implemented it in sreg and chain extensions
* added extension processing to basic_op
* added sreg to test OP
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
I shouldn't be failing if there's no claimed_id field for OpenID 1 request.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
escaping string for inclusion into x(ht)ml attributes
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
generatlized checkauth_message_proxy and added it to util namespace. To be
later used for constructing setup url in 1.0 checkid_immediate reply.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
* added mode_unknown to the mode_t enumeration
* add to_keyvalues(ostream&) to openid message class
* fixed openid_message_t::set_field for the case of overwriting
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Reject associate replies returning secret of inconsistent with association type
length. This way severely broken OPs which return SHA1 association as SHA256
will still work in dumb mode.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
This is the right thing to do and may come in handy assisting OPs in improving
UI.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
This reverts commit 7bde7f66284b47a75bbceadc360e7f03550ace21.
because some older expat version do not support this function I don't want to
make it conditional for now.
Conflicts:
lib/discovery.cc
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
somehow I've managed to put too many tidying options into conditional.
Signed-off-by: Michael Krelin <hacker@klever.net>
|
|
now when parsing document that we expect might be html we also save first 16K
of the document to the buffer and if the parser choked we run the saved data
through htmltidy and feed the output to the parser again.
Signed-off-by: Michael Krelin <hacker@klever.net>
|