PILLUTLAAVINASH / google-enterprise-connector-manager

Automatically exported from code.google.com/p/google-enterprise-connector-manager
0 stars 0 forks source link

GsaFeedConnection.sendData does not handle exceptions. #5

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Run QueryTraverser over a repository using GsaFeedConnection, e.g., by
crawling the repository from the GSA.
2. Return a google:content property with a stream that throws exceptions
during reading. Almost any stream might do this. A simple FilterInputStream
could be written that does the same thing.

What is the expected output? What do you see instead?

I'm not sure how it should be handled, but the take method does not handle
exceptions, so the feed URL preamble is written to the URLConnection's
output stream and then the URLConnection is abandoned when in.read throws
an exception.

Please use labels and text to provide additional information.

I don't know if it's better for the GSA to get the partial feed URL and do
something, but it seems at a minimum like the URLConnection's output stream
should be closed and the response read. 

Perhaps there is some way to indicate in the feed URL that the document was
only partially read and should be ignored?

Original issue reported on code.google.com by jl1615@gmail.com on 19 Jan 2007 at 9:15

GoogleCodeExporter commented 8 years ago
The "take" method is a typo. The correct method name is sendData.

Original comment by jl1615@gmail.com on 19 Jan 2007 at 9:27

GoogleCodeExporter commented 8 years ago
Brian, please add to backlog.

Original comment by donald.z...@gmail.com on 24 Jan 2007 at 11:47

GoogleCodeExporter commented 8 years ago
Brian, please add to backlog.

Original comment by donald.z...@gmail.com on 24 Jan 2007 at 11:47

GoogleCodeExporter commented 8 years ago
Google Bug #243977

Original comment by vjo...@gmail.com on 9 Feb 2007 at 4:04

GoogleCodeExporter commented 8 years ago
Fixed in R450

Original comment by mgron...@gmail.com on 3 Oct 2007 at 9:26

GoogleCodeExporter commented 8 years ago

Original comment by mgron...@gmail.com on 3 Oct 2007 at 9:30

GoogleCodeExporter commented 8 years ago

Original comment by mgron...@gmail.com on 3 Oct 2007 at 10:48