php-java-bridge-users

[Php-java-bridge-users] Proper cleanup in FastCGIServlet

[<php...@li...>]

Hei

I have discovered a "bug" in the addHeader method of FastCGIServlet. It
wrongly asumes that the string it gets after stripping off "Status:" is
a number. I am using it with Joomla and the string that is used in the
line

	response.setStatus(Integer.parseInt(line)); 

contains "403:" - This means that the above line will throw a
NumberFormatException.

Below is a suggested fix for cleaning up the "line" prior to setting the
status.

	while(!line.matches("^\\d*$") && line.length()>0)
	{
		line=line.substr(0,line.length-1);
	}
	if(line.length()>0)
	{
		response.setStatus(Integer.parseInt(line)); 	
	}
	else
	{
		//Could not deduce a number - asume 200 Ok
		response.setStatus(200);
	}


Regards
Hermod
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *

This email with attachments is solely for the use of the individual or
entity to whom it is addressed. Please also be aware that the DnB NOR Group
cannot accept any payment orders or other legally binding correspondence with
customers as a part of an email. 

This email message has been virus checked by the anti virus programs used
in the DnB NOR Group.

* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *

Re: [Php-java-bridge-users] Proper cleanup in FastCGIServlet

[<php...@li...>]

Hi,

403:

is not a valid status code.

Valid codes are 200 OK or 418 I'm a teapot. 403: is just garbage.

Regards,
Jost Bökemeier

Re: [Php-java-bridge-users] Proper cleanup in FastCGIServlet

[<php...@li...>]

Hei

Come on! Assuming that any string is a number is wrong. This is the servlet splitting as string into pieces and trying to get a number from it. Whoe's at fault here does not matter. It is always better to be safe than sorry.

Merry Christmas.

Regards,
Hermod 

-----Original Message-----
From: php...@li... [mailto:php...@li...] 
Sent: Monday, December 20, 2010 8:50 PM
To: php...@li...
Subject: Re: [Php-java-bridge-users] Proper cleanup in FastCGIServlet

Hi,

403:

is not a valid status code.

Valid codes are 200 OK or 418 I'm a teapot. 403: is just garbage.

Regards,
Jost Bökemeier
------------------------------------------------------------------------------
Lotusphere 2011
Register now for Lotusphere 2011 and learn how to connect the dots, take your collaborative environment to the next level, and enter the era of Social Business.
http://p.sf.net/sfu/lotusphere-d2d
_______________________________________________
php-java-bridge-users mailing list
php...@li...
https://lists.sourceforge.net/lists/listinfo/php-java-bridge-users
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *

This email with attachments is solely for the use of the individual or
entity to whom it is addressed. Please also be aware that the DnB NOR Group
cannot accept any payment orders or other legally binding correspondence with
customers as a part of an email. 

This email message has been virus checked by the anti virus programs used
in the DnB NOR Group.

* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *