signal an error if discovery discovers nothing
|Reported by:||avodonosov||Owned by:|
If discovery process doesn't discover the necessary information (provider endpoint at least, but we should check by the spec the minimum required attribute list), the function should signal an error, instead of leaving NILs in the AUTHPROC structure, and allow the library fail later (which is inconvenient to debug).
It's necessary to add a check to the end of the discovery process, that verifies that the required minimum of information is discovered.
I encountered this problem when was recently testing with technorati.com - they previously were providing OpenID, and I registered an account.
The OpenID identity URL is http://technorati.com/people/technorati/clopenid. Today this page does not have the <link> tags necessary for discovery. In result, the discovery process just silently created an AUTHPROC struct with NILs. The error occurred somewhere later in the control flow.
Change History (4)
comment:2 Changed 3 years ago by avodonosov
- Summary changed from signal an error if discovery didn't discovered anything to signal an error if discovery discovers nothing
comment:3 follow-up: ↓ 4 Changed 3 years ago by avodonosov
- Resolution set to fixed
- Status changed from new to closed