diff options
| author | Andrew Arnott <andrewarnott@gmail.com> | 2009-06-01 17:14:12 -0700 |
|---|---|---|
| committer | Andrew Arnott <andrewarnott@gmail.com> | 2009-06-02 16:58:28 -0700 |
| commit | 507fe96dc2835078ac649bcc36395275f4736217 (patch) | |
| tree | b1350dff1e3f0ecaf8f040cfb2d05daac21f9abd /src | |
| parent | a054f5d3eaf59f4f144753d065ea4f73e42acf16 (diff) | |
| download | DotNetOpenAuth-507fe96dc2835078ac649bcc36395275f4736217.zip DotNetOpenAuth-507fe96dc2835078ac649bcc36395275f4736217.tar.gz DotNetOpenAuth-507fe96dc2835078ac649bcc36395275f4736217.tar.bz2 | |
Added log message to explain why no services are discovered with RequireSsl on none-SSL identifier.
Diffstat (limited to 'src')
| -rw-r--r-- | src/DotNetOpenAuth/OpenId/RelyingParty/AuthenticationRequest.cs | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/DotNetOpenAuth/OpenId/RelyingParty/AuthenticationRequest.cs b/src/DotNetOpenAuth/OpenId/RelyingParty/AuthenticationRequest.cs index 32a6a83..25c4a15 100644 --- a/src/DotNetOpenAuth/OpenId/RelyingParty/AuthenticationRequest.cs +++ b/src/DotNetOpenAuth/OpenId/RelyingParty/AuthenticationRequest.cs @@ -271,7 +271,10 @@ namespace DotNetOpenAuth.OpenId.RelyingParty { if (relyingParty.SecuritySettings.RequireSsl) { // Rather than check for successful SSL conversion at this stage, // We'll wait for secure discovery to fail on the new identifier. - userSuppliedIdentifier.TryRequireSsl(out userSuppliedIdentifier); + if (!userSuppliedIdentifier.TryRequireSsl(out userSuppliedIdentifier)) { + // But at least log the failure. + Logger.OpenId.WarnFormat("RequireSsl mode is on, so discovery on insecure identifier {0} will yield no results.", userSuppliedIdentifier); + } } if (Logger.OpenId.IsWarnEnabled && returnToUrl.Query != null) { |