GERONIMO-4252 NNTP Transport provider does not correctly store message identifiers
Patch provided by David Graf
git-svn-id: https://svn.apache.org/repos/asf/geronimo/javamail/trunk@686231 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/geronimo-javamail_1.4/geronimo-javamail_1.4_provider/src/main/java/org/apache/geronimo/javamail/store/nntp/NNTPGroupFolder.java b/geronimo-javamail_1.4/geronimo-javamail_1.4_provider/src/main/java/org/apache/geronimo/javamail/store/nntp/NNTPGroupFolder.java
index 1c3941f..aa01696 100644
--- a/geronimo-javamail_1.4/geronimo-javamail_1.4_provider/src/main/java/org/apache/geronimo/javamail/store/nntp/NNTPGroupFolder.java
+++ b/geronimo-javamail_1.4/geronimo-javamail_1.4_provider/src/main/java/org/apache/geronimo/javamail/store/nntp/NNTPGroupFolder.java
@@ -238,7 +238,9 @@
int idStart = response.indexOf('<');
int idEnd = response.indexOf('>');
- message = new NNTPMessage(this, (NNTPStore) store, msgNum, response.substring(idStart + 1, idEnd));
+ // NB: The "<" and ">" delimiters are required elements of the message id, not just
+ // delimiters for the sake of the command. We need to keep these around
+ message = new NNTPMessage(this, (NNTPStore) store, msgNum, response.substring(idStart, idEnd + 1));
// add this to the article cache.
articles.put(key, message);