diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/DotNetOpenAuth.Test/OpenId/RelyingParty/RelyingPartySecuritySettingsTests.cs | 18 | ||||
| -rw-r--r-- | src/DotNetOpenAuth/OpenId/RelyingParty/RelyingPartySecuritySettings.cs | 34 |
2 files changed, 1 insertions, 51 deletions
diff --git a/src/DotNetOpenAuth.Test/OpenId/RelyingParty/RelyingPartySecuritySettingsTests.cs b/src/DotNetOpenAuth.Test/OpenId/RelyingParty/RelyingPartySecuritySettingsTests.cs index cb5fbb5..3270a9f 100644 --- a/src/DotNetOpenAuth.Test/OpenId/RelyingParty/RelyingPartySecuritySettingsTests.cs +++ b/src/DotNetOpenAuth.Test/OpenId/RelyingParty/RelyingPartySecuritySettingsTests.cs @@ -35,23 +35,5 @@ namespace DotNetOpenAuth.Test.OpenId.RelyingParty { this.settings.RequireSsl = false; Assert.IsFalse(this.settings.RequireSsl); } - - /// <summary> - /// Verifies that changing the <see cref="RelyingPartySecuritySettings.RequireSsl"/> property - /// fires the <see cref="RelyingPartySecuritySettings.RequireSslChanged"/> event. - /// </summary> - [TestMethod] - public void RequireSslFiresEvent() { - bool requireSslChanged = false; - this.settings.RequireSslChanged += (sender, e) => { requireSslChanged = true; }; - - // Setting the property to its current value should not fire event. - this.settings.RequireSsl = this.settings.RequireSsl; - Assert.IsFalse(requireSslChanged); - - // Changing the property's value should fire the event. - this.settings.RequireSsl = !this.settings.RequireSsl; - Assert.IsTrue(requireSslChanged); - } } } diff --git a/src/DotNetOpenAuth/OpenId/RelyingParty/RelyingPartySecuritySettings.cs b/src/DotNetOpenAuth/OpenId/RelyingParty/RelyingPartySecuritySettings.cs index f7ac3c2..cc786dd 100644 --- a/src/DotNetOpenAuth/OpenId/RelyingParty/RelyingPartySecuritySettings.cs +++ b/src/DotNetOpenAuth/OpenId/RelyingParty/RelyingPartySecuritySettings.cs @@ -13,11 +13,6 @@ namespace DotNetOpenAuth.OpenId.RelyingParty { /// </summary> public sealed class RelyingPartySecuritySettings : SecuritySettings { /// <summary> - /// Backing field for the <see cref="RequireSsl"/> property. - /// </summary> - private bool requireSsl; - - /// <summary> /// Initializes a new instance of the <see cref="RelyingPartySecuritySettings"/> class. /// </summary> internal RelyingPartySecuritySettings() @@ -26,11 +21,6 @@ namespace DotNetOpenAuth.OpenId.RelyingParty { } /// <summary> - /// Fired when the <see cref="RequireSsl"/> property is changed. - /// </summary> - internal event EventHandler RequireSslChanged; - - /// <summary> /// Gets or sets a value indicating whether the entire pipeline from Identifier discovery to /// Provider redirect is guaranteed to be encrypted using HTTPS for authentication to succeed. /// </summary> @@ -58,19 +48,7 @@ namespace DotNetOpenAuth.OpenId.RelyingParty { /// A <see cref="ProtocolException"/> is thrown during discovery or authentication when a secure pipeline cannot be established. /// </para> /// </remarks> - public bool RequireSsl { - get { - return this.requireSsl; - } - - set { - if (this.requireSsl == value) { - return; - } - this.requireSsl = value; - this.OnRequireSslChanged(); - } - } + public bool RequireSsl { get; set; } /// <summary> /// Gets or sets the oldest version of OpenID the remote party is allowed to implement. @@ -84,15 +62,5 @@ namespace DotNetOpenAuth.OpenId.RelyingParty { /// </summary> /// <value>The default value is 7 days.</value> public TimeSpan PrivateSecretMaximumAge { get; set; } - - /// <summary> - /// Fires the <see cref="RequireSslChanged"/> event. - /// </summary> - private void OnRequireSslChanged() { - EventHandler requireSslChanged = this.RequireSslChanged; - if (requireSslChanged != null) { - requireSslChanged(this, new EventArgs()); - } - } } } |