in topic.asp you find this code:
Member_ID = getNewMemberNumber()
if (strAuthType = "nt") then
set my_Conn = Server.CreateObject("ADODB.Connection")
my_Conn.Open strConnString
call NTauthenticate()
if (ChkAccountReg() = "1") then
call NTUser()
end if
end if
%>
<!--#INCLUDE FILE="inc_top.asp" -->
<%
with a few problems:
1) an error message because getNewMemberNumber() needs an open connection
2) there is another connection opened and never closed
The initial reason to put part of the code before the inc_top.asp is because NTUser() writes cookies, but because we use Response.buffer = TRUE this isn't a problem anymore.
The getNewMemberNumber() has to be moved after the inc_top.asp anyhow, but I would suggest changing the code to:
%>
<!--#INCLUDE FILE="inc_top.asp" -->
<%
Member_ID = getNewMemberNumber()
if (strAuthType = "nt") then
set my_Conn = Server.CreateObject("ADODB.Connection")
my_Conn.Open strConnString
call NTauthenticate()
if (ChkAccountReg() = "1") then
call NTUser()
end if
end if
thus removing one, not needed, connection to the database.
Pierre
Forum Locked
Topic Locked
Posted - 18 February 2001 : 06:27:19
