fixed various bugs (see SF tracker)

obiltschnig · obiltschnig · commit 57762a2cfcf7 · 2008-09-16T17:26:28.000Z
diff --git a/Foundation/Foundation_vs71.vcproj b/Foundation/Foundation_vs71.vcproj
@@ -164,6 +164,7 @@
 				Name="VCCustomBuildTool"/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mtd.lib"/>
 			<Tool
 				Name="VCMIDLTool"/>
@@ -217,6 +218,7 @@
 				Name="VCCustomBuildTool"/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mt.lib"/>
 			<Tool
 				Name="VCMIDLTool"/>
diff --git a/Foundation/Foundation_vs80.vcproj b/Foundation/Foundation_vs80.vcproj
@@ -256,6 +256,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mt.lib"
 			/>
 			<Tool
@@ -327,6 +328,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mtd.lib"
 			/>
 			<Tool
diff --git a/Foundation/Foundation_vs90.vcproj b/Foundation/Foundation_vs90.vcproj
@@ -249,6 +249,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mt.lib"
 			/>
 			<Tool
@@ -318,6 +319,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mtd.lib"
 			/>
 			<Tool
diff --git a/Foundation/include/Poco/LinearHashTable.h b/Foundation/include/Poco/LinearHashTable.h
@@ -89,7 +89,7 @@ class LinearHashTable
 	typedef typename Bucket::iterator    BucketIterator;
 	typedef typename BucketVec::iterator BucketVecIterator;
 
-	class ConstIterator
+	class ConstIterator: public std::iterator<std::forward_iterator_tag, Value>
 	{
 	public:
 		ConstIterator()
diff --git a/Foundation/src/AsyncChannel.cpp b/Foundation/src/AsyncChannel.cpp
@@ -115,7 +115,10 @@ void AsyncChannel::close()
 	{
 		while (!_queue.empty()) Thread::sleep(100);
 		
-		do { _queue.wakeUpAll(); }
+		do 
+		{
+			_queue.wakeUpAll(); 
+		}
 		while (!_thread.tryJoin(100));
 	}
 }
diff --git a/Net/Net_vs71.vcproj b/Net/Net_vs71.vcproj
@@ -166,6 +166,7 @@
 				Name="VCCustomBuildTool"/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mtd.lib"/>
 			<Tool
 				Name="VCMIDLTool"/>
@@ -219,6 +220,7 @@
 				Name="VCCustomBuildTool"/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mt.lib"/>
 			<Tool
 				Name="VCMIDLTool"/>
diff --git a/Net/Net_vs80.vcproj b/Net/Net_vs80.vcproj
@@ -258,6 +258,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mt.lib"
 			/>
 			<Tool
@@ -329,6 +330,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mtd.lib"
 			/>
 			<Tool
diff --git a/Net/Net_vs90.vcproj b/Net/Net_vs90.vcproj
@@ -249,6 +249,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mt.lib"
 			/>
 			<Tool
@@ -318,6 +319,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\Poco$(ProjectName)mtd.lib"
 			/>
 			<Tool
diff --git a/Net/include/Poco/Net/HTTPCookie.h b/Net/include/Poco/Net/HTTPCookie.h
@@ -1,7 +1,7 @@
 //
 // HTTPCookie.h
 //
-// $Id: //poco/svn/Net/include/Poco/Net/HTTPCookie.h#2 $
+// $Id: //poco/1.3/Net/include/Poco/Net/HTTPCookie.h#1 $
 //
 // Library: Net
 // Package: HTTP
@@ -157,6 +157,12 @@ class Net_API HTTPCookie
 		/// Returns the maximum age in seconds for
 		/// the cookie.
 		
+	void setHttpOnly(bool flag = true);
+		/// Sets the HttpOnly flag for the cookie.
+		
+	bool getHttpOnly() const;
+		/// Returns true iff the cookie's HttpOnly flag is set.
+		
 	std::string toString() const;
 		/// Returns a string representation of the cookie,
 		/// suitable for use in a Set-Cookie header.
@@ -170,6 +176,7 @@ class Net_API HTTPCookie
 	std::string _path;
 	bool        _secure;
 	int         _maxAge;
+	bool        _httpOnly;
 };
 
 
@@ -224,6 +231,12 @@ inline int HTTPCookie::getMaxAge() const
 }
 
 
+inline bool HTTPCookie::getHttpOnly() const
+{
+	return _httpOnly;
+}
+
+
 } } // namespace Poco::Net
 
 
diff --git a/Net/src/HTMLForm.cpp b/Net/src/HTMLForm.cpp
@@ -124,6 +124,7 @@ void HTMLForm::addPart(const std::string& name, PartSource* pSource)
 
 void HTMLForm::load(const HTTPRequest& request, std::istream& requestBody, PartHandler& handler)
 {
+	clear();
 	if (request.getMethod() == HTTPRequest::HTTP_POST)
 	{
 		std::string mediaType;
diff --git a/Net/src/HTTPCookie.cpp b/Net/src/HTTPCookie.cpp
@@ -1,7 +1,7 @@
 //
 // HTTPCookie.cpp
 //
-// $Id: //poco/svn/Net/src/HTTPCookie.cpp#2 $
+// $Id: //poco/1.3/Net/src/HTTPCookie.cpp#2 $
 //
 // Library: Net
 // Package: HTTP
@@ -63,7 +63,8 @@ namespace Net {
 HTTPCookie::HTTPCookie():
 	_version(0),
 	_secure(false),
-	_maxAge(-1)
+	_maxAge(-1),
+	_httpOnly(false)
 {
 }
 
@@ -72,15 +73,17 @@ HTTPCookie::HTTPCookie(const std::string& name):
 	_version(0),
 	_name(name),
 	_secure(false),
-	_maxAge(-1)
+	_maxAge(-1),
+	_httpOnly(false)
 {
 }
 
 
 HTTPCookie::HTTPCookie(const NameValueCollection& nvc):
 	_version(0),
 	_secure(false),
-	_maxAge(-1)
+	_maxAge(-1),
+	_httpOnly(false)
 {
 	for (NameValueCollection::ConstIterator it = nvc.begin(); it != nvc.end(); ++it)
 	{
@@ -117,6 +120,10 @@ HTTPCookie::HTTPCookie(const NameValueCollection& nvc):
 		{
 			setVersion(NumberParser::parse(value));
 		}
+		else if (icompare(name, "HttpOnly") == 0)
+		{
+			setHttpOnly(true);
+		}
 		else
 		{
 			setName(name);
@@ -131,7 +138,8 @@ HTTPCookie::HTTPCookie(const std::string& name, const std::string& value):
 	_name(name),
 	_value(value),
 	_secure(false),
-	_maxAge(-1)
+	_maxAge(-1),
+	_httpOnly(false)
 {
 }
 
@@ -144,7 +152,8 @@ HTTPCookie::HTTPCookie(const HTTPCookie& cookie):
 	_domain(cookie._domain),
 	_path(cookie._path),
 	_secure(cookie._secure),
-	_maxAge(cookie._maxAge)
+	_maxAge(cookie._maxAge),
+	_httpOnly(cookie._httpOnly)
 {
 }
 
@@ -158,14 +167,15 @@ HTTPCookie& HTTPCookie::operator = (const HTTPCookie& cookie)
 {
 	if (&cookie != this)
 	{
-		_version = cookie._version;
-		_name    = cookie._name;
-		_value   = cookie._value;
-		_comment = cookie._comment;
-		_domain  = cookie._domain;
-		_path    = cookie._path;
-		_secure  = cookie._secure;
-		_maxAge  = cookie._maxAge;
+		_version  = cookie._version;
+		_name     = cookie._name;
+		_value    = cookie._value;
+		_comment  = cookie._comment;
+		_domain   = cookie._domain;
+		_path     = cookie._path;
+		_secure   = cookie._secure;
+		_maxAge   = cookie._maxAge;
+		_httpOnly = cookie._httpOnly;
 	}
 	return *this;
 }
@@ -219,6 +229,12 @@ void HTTPCookie::setMaxAge(int maxAge)
 }
 
 
+void HTTPCookie::setHttpOnly(bool flag)
+{
+	_httpOnly = flag;
+}
+
+
 std::string HTTPCookie::toString() const
 {
 	std::string result(_name);
@@ -248,6 +264,10 @@ std::string HTTPCookie::toString() const
 		{
 			result.append("; secure");
 		}
+		if (_httpOnly)
+		{
+			result.append("; HttpOnly");
+		}
 	}
 	else
 	{
@@ -283,6 +303,10 @@ std::string HTTPCookie::toString() const
 		{
 			result.append("; secure");
 		}
+		if (_httpOnly)
+		{
+			result.append("; HttpOnly");
+		}
 		result.append("; Version=\"1\"");
 	}
 	return result;
diff --git a/Net/testsuite/src/HTTPCookieTest.cpp b/Net/testsuite/src/HTTPCookieTest.cpp
@@ -1,7 +1,7 @@
 //
 // HTTPCookieTest.cpp
 //
-// $Id: //poco/svn/Net/testsuite/src/HTTPCookieTest.cpp#2 $
+// $Id: //poco/1.3/Net/testsuite/src/HTTPCookieTest.cpp#1 $
 //
 // Copyright (c) 2005-2006, Applied Informatics Software Engineering GmbH.
 // and Contributors.
@@ -63,13 +63,19 @@ void HTTPCookieTest::testCookie()
 	assert (cookie.toString() == "name=value; domain=appinf.com; path=/");
 	cookie.setSecure(true);
 	assert (cookie.toString() == "name=value; domain=appinf.com; path=/; secure");
+	cookie.setHttpOnly(true);
+	assert (cookie.toString() == "name=value; domain=appinf.com; path=/; secure; HttpOnly");
+	cookie.setHttpOnly(false);
 	
 	cookie.setVersion(1);
 	assert (cookie.toString() == "name=\"value\"; Comment=\"comment\"; Domain=\"appinf.com\"; Path=\"/\"; secure; Version=\"1\"");
 	
 	cookie.setSecure(false);
 	cookie.setMaxAge(100);
 	assert (cookie.toString() == "name=\"value\"; Comment=\"comment\"; Domain=\"appinf.com\"; Path=\"/\"; Max-Age=\"100\"; Version=\"1\"");
+	
+	cookie.setHttpOnly(true);
+	assert (cookie.toString() == "name=\"value\"; Comment=\"comment\"; Domain=\"appinf.com\"; Path=\"/\"; Max-Age=\"100\"; HttpOnly; Version=\"1\"");
 }
 
 
diff --git a/NetSSL_OpenSSL/NetSSL_OpenSSL_vs71.vcproj b/NetSSL_OpenSSL/NetSSL_OpenSSL_vs71.vcproj
@@ -166,6 +166,7 @@
 				Name="VCCustomBuildTool"/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\PocoNetSSLmtd.lib"/>
 			<Tool
 				Name="VCMIDLTool"/>
@@ -219,6 +220,7 @@
 				Name="VCCustomBuildTool"/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\PocoNetSSLmt.lib"/>
 			<Tool
 				Name="VCMIDLTool"/>
diff --git a/NetSSL_OpenSSL/NetSSL_OpenSSL_vs80.vcproj b/NetSSL_OpenSSL/NetSSL_OpenSSL_vs80.vcproj
@@ -254,6 +254,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\PocoNetSSLmtd.lib"
 			/>
 			<Tool
@@ -329,6 +330,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\PocoNetSSLmt.lib"
 			/>
 			<Tool
diff --git a/NetSSL_OpenSSL/NetSSL_OpenSSL_vs90.vcproj b/NetSSL_OpenSSL/NetSSL_OpenSSL_vs90.vcproj
@@ -245,6 +245,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\PocoNetSSLmtd.lib"
 			/>
 			<Tool
@@ -318,6 +319,7 @@
 			/>
 			<Tool
 				Name="VCLibrarianTool"
+				AdditionalDependencies="ws2_32.lib"
 				OutputFile="..\lib\PocoNetSSLmt.lib"
 			/>
 			<Tool
diff --git a/XML/include/Poco/XML/XMLString.h b/XML/include/Poco/XML/XMLString.h
diff --git a/XML/src/AttributesImpl.cpp b/XML/src/AttributesImpl.cpp
diff --git a/XML/src/NamespaceStrategy.cpp b/XML/src/NamespaceStrategy.cpp
diff --git a/XML/src/ParserEngine.cpp b/XML/src/ParserEngine.cpp

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ class LinearHashTable`
`89`	`89`	`typedef typename Bucket::iterator BucketIterator;`
`90`	`90`	`typedef typename BucketVec::iterator BucketVecIterator;`
`91`	`91`
`92`		`- class ConstIterator`
	`92`	`+ class ConstIterator: public std::iterator<std::forward_iterator_tag, Value>`
`93`	`93`	`{`
`94`	`94`	`public:`
`95`	`95`	`ConstIterator()`
Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,10 @@ void AsyncChannel::close()`
`115`	`115`	`{`
`116`	`116`	`while (!_queue.empty()) Thread::sleep(100);`
`117`	`117`
`118`		`- do { _queue.wakeUpAll(); }`
	`118`	`+ do`
	`119`	`+ {`
	`120`	`+ _queue.wakeUpAll();`
	`121`	`+ }`
`119`	`122`	`while (!_thread.tryJoin(100));`
`120`	`123`	`}`
`121`	`124`	`}`
Original file line number	Diff line number	Diff line change
`@@ -124,6 +124,7 @@ void HTMLForm::addPart(const std::string& name, PartSource* pSource)`
`124`	`124`
`125`	`125`	`void HTMLForm::load(const HTTPRequest& request, std::istream& requestBody, PartHandler& handler)`
`126`	`126`	`{`
	`127`	`+ clear();`
`127`	`128`	`if (request.getMethod() == HTTPRequest::HTTP_POST)`
`128`	`129`	`{`
`129`	`130`	`std::string mediaType;`