search

DSpace / xoai

OAI-PMH Java Toolkit
29 stars 52 forks source link

Close stream in ListSetsHandler #52

Closed cgendreau closed 7 years ago

cgendreau commented 9 years ago

ListSetsHandler should close his stream otherwise it will result in a java.lang.IllegalStateException: Invalid use of BasicClientConnManager: connection still allocated. when used with the HttpOAIClient and the sets are returned over more than 1 page.

Abdull commented 8 years ago

Looks good to me. Can this pull request and #53 be merged?

mmalmeida commented 8 years ago

@Abdull there is a developer's meeting next week and we'll discuss these merges.

mmalmeida commented 7 years ago

@cgendreau would you be able to provide a test case for this change?

Thanks!