ThreadAbort errors with new spambot prevention for /register

Description

Line 756 of Register.ascx.cs has a redirect that terminates the response which results in a ThreadAbort exception. The impact is minimal, except it creates a lot of unnecessary noise in the event log.

AssemblyVersion:7.3.2
PortalID:49
PortalName:AAG Insurance
UserID:-1
UserName:
ActiveTabID:4945
ActiveTabName:Home
RawURL:/register
AbsoluteURL:/Default.aspx
AbsoluteURLReferrer:http://www.aag-services.com/
UserAgent:Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/28.0.1500.72 Safari/537.36
DefaultDataProviderotNetNuke.Data.SqlDataProvider, DotNetNuke
ExceptionGUID:d47ec9c8-3b9b-48a9-bff5-f5ab756de806
InnerException:Unhandled Error Adding Module to ContentPane
FileName:
FileLineNumber:0
FileColumnNumber:0
Method:System.Threading.Thread.AbortInternal
StackTrace:
Message:
DotNetNuke.Services.Exceptions.ModuleLoadException: Unhandled Error Adding Module to ContentPane ---> System.Threading.ThreadAbortException: Thread was being aborted.
at System.Threading.Thread.AbortInternal()
at System.Threading.Thread.Abort(Object stateInfo)
at System.Web.HttpResponse.AbortCurrentThread()
at DotNetNuke.Modules.Admin.Users.Register.OnInit(EventArgs e) in c:\Websites\Agency Revolution\DesktopModules\Admin\Security\Register.ascx.cs:line 756
at System.Web.UI.Control.InitRecursive(Control namingContainer)
at System.Web.UI.Control.InitRecursive(Control namingContainer)
at System.Web.UI.Control.AddedControl(Control control, Int32 index)
at System.Web.UI.Control.EnsureChildControls()
at DotNetNuke.UI.Containers.Container.get_ModuleControl()
at DotNetNuke.UI.Containers.Container.ProcessModule()
at DotNetNuke.UI.Skins.Pane.InjectModule(ModuleInfo module)
— End of inner exception stack trace —
at DotNetNuke.UI.Skins.Pane.InjectModule(ModuleInfo module)
at DotNetNuke.UI.Skins.Skin.InjectModule(Pane pane, ModuleInfo module)
Source:

QA Test Plan

None

Activity

Show:
Ken Grierson
September 15, 2014, 3:12 PM

I have verified the code in in place in 7.3.3 build 95
I set the Registration to None and append /register to a site URL which generates a log in page and no exception is logged.
This is the correct behavior

Complete

Assignee

Ken Grierson

Reporter

Lucas Jans

Story Size

Unknown

Severity

Trivial

Triage

New

Reported in Build #

None

Fixed in Build

Dev Owner

None

Includes Code Fix

Yes

Documentation Required

No

Trouble Ticket

None

Requires More Info

None

QA Story Points

None

QA Owner

None

Injected

None

Automation Required

None

Code Review Owner

None

Components

Fix versions

Affects versions

Priority

High