ProductCart Shopping Cart Software Forums : 301 Redirect https://forum.productcart.com/ Copyright (c) 2006-2013 Web Wiz Forums - All Rights Reserved. Wed, 08 Apr 2026 13:54:29 +0000 Thu, 19 Aug 2010 14:38:05 +0000 http://blogs.law.harvard.edu/tech/rss Web Wiz Forums 12.04 360 https://forum.productcart.com/RSS_post_feed.asp?TID=3308 <![CDATA[ProductCart Shopping Cart Software Forums]]> https://forum.productcart.com/forum_images/pc_logo_50.png https://forum.productcart.com/ <![CDATA[301 Redirect : Additional question to this...]]> https://forum.productcart.com/301-redirect_topic3308_post15301.html#15301 Author: worldofrugs
Subject: 3308
Posted: 19-August-2010 at 2:38pm

Additional question to this issue...
While the given code works perfectly (thanks Greg!), I'm wondering if the following would be an option.

Say I have 5 categories: 120/121/122/123/124
I could write the code for each single line, but could it be made more simple by doing something like:
If idCategory >119 AND idCategory < 125 Then
.... [action to be taken] .....

I know I prob. have to convert the string to a number, but not sure on these things.
Somehow I have a feeling this should be simple to do?

It sure would save a lot of coding!

------ EDIT ------
Found the solution Wink

Dim idCategory
idCategory = Cint(getUserInput(Request("idCategory"),5))
if idCategory>119 AND idCategory<125 then
.... [action to be taken] .....


Edited by worldofrugs - 19-August-2010 at 2:58pm]]> Thu, 19 Aug 2010 14:38:05 +0000 https://forum.productcart.com/301-redirect_topic3308_post15301.html#15301 <![CDATA[301 Redirect : I was responding to the code he...]]> https://forum.productcart.com/301-redirect_topic3308_post14635.html#14635 Author: Greg Dinger
Subject: 3308
Posted: 03-June-2010 at 2:50pm

I was responding to the code he posted and wanted to point out the importance that when he is dealing directly with querstring variables the need for caution.]]> Thu, 03 Jun 2010 14:50:15 +0000 https://forum.productcart.com/301-redirect_topic3308_post14635.html#14635 <![CDATA[301 Redirect : Isn&#039;t the native ProductCart...]]> https://forum.productcart.com/301-redirect_topic3308_post14631.html#14631 Author: Brett
Subject: 3308
Posted: 03-June-2010 at 1:45pm

Isn't the native ProductCart variable pIdCategory category already sanitized?

*edit*

Not to confuse anyone... Greg's solution is completely secure and I'm sure the processor overhead for such a simple calculation is minimal. However, it seems a bit redundant to have two variables holding the same number.

Edited by Brett - 03-June-2010 at 2:18pm]]> Thu, 03 Jun 2010 13:45:08 +0000 https://forum.productcart.com/301-redirect_topic3308_post14631.html#14631 <![CDATA[301 Redirect : Thanks for the help Greg. I&#039;m...]]> https://forum.productcart.com/301-redirect_topic3308_post14629.html#14629 Author: worldofrugs
Subject: 3308
Posted: 03-June-2010 at 10:39am

Thanks for the help Greg. I'm implementing the pages that need to be re-directed right now and it works as it should.
It's a good feeling to know that I'm "safe" now
Thanks!!!
]]> Thu, 03 Jun 2010 10:39:35 +0000 https://forum.productcart.com/301-redirect_topic3308_post14629.html#14629 <![CDATA[301 Redirect : That&#039;s what I had in mind....]]> https://forum.productcart.com/301-redirect_topic3308_post14628.html#14628 Author: Greg Dinger
Subject: 3308
Posted: 03-June-2010 at 10:33am

That's what I had in mind.  You DO NOT want to wake up one day and receive reports that the store has been attacked.  Concientious use of proper protection methods is absolutely critical.]]> Thu, 03 Jun 2010 10:33:36 +0000 https://forum.productcart.com/301-redirect_topic3308_post14628.html#14628 <![CDATA[301 Redirect : Never mind Greg, I found the...]]> https://forum.productcart.com/301-redirect_topic3308_post14627.html#14627 Author: worldofrugs
Subject: 3308
Posted: 03-June-2010 at 10:30am

Never mind Greg, I found the issue.
On the Wiki page you gave me, you have the line:
Dim idCategory
idCategory = getUserInput(Request("id"),5)


But it should actually be:
Dim idCategory
idCategory = getUserInput(Request("idCategory "),5)


When I changed it this way, working great, so I assume that this is the best way to go?
Thanks for pointing it out Greg, 'you tha man'! Wink


Edited by worldofrugs - 03-June-2010 at 10:31am]]> Thu, 03 Jun 2010 10:30:26 +0000 https://forum.productcart.com/301-redirect_topic3308_post14627.html#14627 <![CDATA[301 Redirect : Thanks for that tip Greg...However,...]]> https://forum.productcart.com/301-redirect_topic3308_post14626.html#14626 Author: worldofrugs
Subject: 3308
Posted: 03-June-2010 at 10:26am

Thanks for that tip Greg...
However, as I am not a programming guru, I'm not sure how to implement this in my situation.

I have quickly tried:
Dim idCategory
idCategory = getUserInput(Request("id"),5)
if idCategory="224" then
Response.Status="301 Moved Permanently"
Response.AddHeader "Location", "/shop/pc/TheOtherPage.asp"
Response.End()
end if


but this does not work. It simply goes to the "msg.asp?message=86" page (not a valid category), not resulting in the redirect I want and not returning the 301 header status.

Did I miss something?
]]> Thu, 03 Jun 2010 10:26:08 +0000 https://forum.productcart.com/301-redirect_topic3308_post14626.html#14626 <![CDATA[301 Redirect : Paul, on another note, you should...]]> https://forum.productcart.com/301-redirect_topic3308_post14625.html#14625 Author: Greg Dinger
Subject: 3308
Posted: 03-June-2010 at 9:59am

Paul, on another note, you should make it a general practice to sanitize your reference to querystring values.  Whether the resulting value is sent to a query or not, use the "getuserinput" function.  Were you to not do so when sending a  querystring value to a query, you would create a security hold that could be deadly.
 
http://wiki.earlyimpact.com/how_to/sanitize_strings?s[]=getuserinput
 
In your example above, the following statement would cause you massive grief "catnumber" was sent to a query and a hacker discovered it.
 
CatNumber= lcase(Request.ServerVariables("Query_String"))


Edited by Greg Dinger - 03-June-2010 at 9:59am]]> Thu, 03 Jun 2010 09:59:12 +0000 https://forum.productcart.com/301-redirect_topic3308_post14625.html#14625 <![CDATA[301 Redirect : Thanks for that tip Brett! Will...]]> https://forum.productcart.com/301-redirect_topic3308_post14624.html#14624 Author: worldofrugs
Subject: 3308
Posted: 03-June-2010 at 9:50am

Thanks for that tip Brett!  Will try this out today and when it works (asuming it does), it will keep the coding a little cleaner
Thanks!
Clap
]]> Thu, 03 Jun 2010 09:50:52 +0000 https://forum.productcart.com/301-redirect_topic3308_post14624.html#14624 <![CDATA[301 Redirect : You might be able to find the...]]> https://forum.productcart.com/301-redirect_topic3308_post14614.html#14614 Author: Brett
Subject: 3308
Posted: 03-June-2010 at 1:20am

You might be able to find the category by simply doing:

<% if pIdCategory="number you want" then %>

Since viewcategories.asp is already populating that variable]]> Thu, 03 Jun 2010 01:20:11 +0000 https://forum.productcart.com/301-redirect_topic3308_post14614.html#14614