LAST and NEXT selection logic is working.
[citadel.git] / citadel / modules / nntp / serv_nntp.c
index f7e5cb3e6db6e8bad534cc3dbd9010e5ebc8686b..85dc50a31a32467bd6cda3daa14e2509ffea6d36 100644 (file)
@@ -99,7 +99,6 @@ int is_valid_newsgroup_name(char *name) {
 }
 
 
-
 //
 // Convert a Citadel room name to a valid newsgroup name
 //
@@ -195,9 +194,7 @@ void nntp_greeting(void)
                return;
        }
 
-       // Note: the FQDN *must* appear as the first thing after the 220 code.
-       // Some clients (including citmail.c) depend on it being there.
-       //
+       // Display the standard greeting
        cprintf("200 %s NNTP Citadel server is not finished yet\r\n", config.c_fqdn);
 }
 
@@ -214,7 +211,6 @@ void nntps_greeting(void) {
 }
 
 
-
 //
 // implements the STARTTLS command
 //
@@ -271,7 +267,6 @@ void nntp_cleanup(void)
 }
 
 
-
 //
 // Implements the AUTHINFO USER command (RFC 4643)
 //
@@ -323,7 +318,6 @@ void nntp_authinfo_pass(const char *buf)
 }
 
 
-
 //
 // Implements the AUTHINFO extension (RFC 4643) in USER/PASS mode
 //
@@ -365,20 +359,6 @@ struct nntp_msglist nntp_fetch_msglist(struct ctdlroom *qrbuf) {
 }
 
 
-
-//
-// Various output formats for the LIST commands
-//
-enum {
-       NNTP_LIST_ACTIVE,
-       NNTP_LIST_ACTIVE_TIMES,
-       NNTP_LIST_DISTRIB_PATS,
-       NNTP_LIST_HEADERS,
-       NNTP_LIST_NEWSGROUPS,
-       NNTP_LIST_OVERVIEW_FMT
-};
-
-
 //
 // Output a room name (newsgroup name) in formats required for LIST and NEWGROUPS command
 //
@@ -419,7 +399,6 @@ void output_roomname_in_list_format(struct ctdlroom *qrbuf, int which_format, ch
 }
 
 
-
 //
 // Called once per room by nntp_newgroups() to qualify and possibly output a single room
 //
@@ -450,13 +429,8 @@ void nntp_newgroups_backend(struct ctdlroom *qrbuf, void *data)
 // Implements the NEWGROUPS command
 //
 void nntp_newgroups(const char *cmd) {
-       /*
-        * HACK: this works because the 5XX series error codes from citadel
-        * protocol will also be considered error codes by an NNTP client
-        */
        if (CtdlAccessCheck(ac_logged_in_or_guest)) return;
 
-
        char stringy_date[16];
        char stringy_time[16];
        char stringy_gmt[16];
@@ -513,10 +487,6 @@ void nntp_list_backend(struct ctdlroom *qrbuf, void *data)
 // Implements the LIST commands
 //
 void nntp_list(const char *cmd) {
-       //
-       // HACK: this works because the 5XX series error codes from citadel
-       // protocol will also be considered error codes by an NNTP client
-       //
        if (CtdlAccessCheck(ac_logged_in_or_guest)) return;
 
        char list_format[64];
@@ -580,10 +550,6 @@ void nntp_listgroup_backend(long msgnum, void *userdata) {
 // Implements the GROUP and LISTGROUP commands
 //
 void nntp_group(const char *cmd) {
-       //
-       // HACK: this works because the 5XX series error codes from citadel
-       // protocol will also be considered error codes by an NNTP client
-       //
        if (CtdlAccessCheck(ac_logged_in_or_guest)) return;
 
        citnntp *nntpstate = (citnntp *) CC->session_specific_data;
@@ -684,16 +650,11 @@ void nntp_mode(const char *cmd) {
 }
 
 
-
 //
 // Implements the ARTICLE, HEAD, BODY, and STAT commands.
 // (These commands all accept the same parameters; they differ only in how they output the retrieved message.)
 //
 void nntp_article(const char *cmd) {
-       /*
-        * HACK: this works because the 5XX series error codes from citadel
-        * protocol will also be considered error codes by an NNTP client
-        */
        if (CtdlAccessCheck(ac_logged_in_or_guest)) return;
 
        citnntp *nntpstate = (citnntp *) CC->session_specific_data;
@@ -705,6 +666,7 @@ void nntp_article(const char *cmd) {
        int must_change_currently_selected_article = 0;
 
        // We're going to store one of these values in the variable 'acmd' so that
+       // we can quickly check later which version of the output we want.
        enum {
                ARTICLE,
                HEAD,
@@ -828,6 +790,107 @@ void nntp_article(const char *cmd) {
 }
 
 
+//
+// The LAST and NEXT commands are so similar that they are handled by a single function.
+//
+void nntp_last_next(const char *cmd) {
+       if (CtdlAccessCheck(ac_logged_in_or_guest)) return;
+
+       citnntp *nntpstate = (citnntp *) CC->session_specific_data;
+       char which_command[16];
+       int acmd = 0;
+
+       // We're going to store one of these values in the variable 'acmd' so that
+       // we can quickly check later which version of the output we want.
+       enum {
+               NNTP_LAST,
+               NNTP_NEXT
+       };
+
+       extract_token(which_command, cmd, 0, ' ', sizeof which_command);
+
+       if (!strcasecmp(which_command, "last")) {
+               acmd = NNTP_LAST;
+       }
+       else if (!strcasecmp(which_command, "next")) {
+               acmd = NNTP_NEXT;
+       }
+       else {
+               cprintf("500 I'm afraid I can't do that.\r\n");
+               return;
+       }
+
+       // ok, here we go ... fetch the msglist so we can figure out our place in the universe
+       struct nntp_msglist nm;
+       int i = 0;
+       long selected_msgnum = 0;
+
+       nm = nntp_fetch_msglist(&CC->room);
+       if ((nm.num_msgs < 0) || (nm.msgnums == NULL)) {
+               cprintf("500 something bad happened\r\n");
+               return;
+       }
+
+       syslog(LOG_DEBUG, "NNTP: last_next num_msgs=%d and current article was %ld", nm.num_msgs, nntpstate->current_article_number);
+
+       if ( (acmd == NNTP_LAST) && (nm.num_msgs == 0) ) {
+                       cprintf("422 no previous article in this group 1\r\n"); // nothing here
+       }
+
+       else if ( (acmd == NNTP_LAST) && (nntpstate->current_article_number <= nm.msgnums[0]) ) {
+                       cprintf("422 no previous article in this group 2\r\n"); // already at the beginning
+       }
+
+       else if (acmd == NNTP_LAST) {
+               for (i=0; ((i<nm.num_msgs)&&(selected_msgnum<=0)); ++i) {
+                       if ( (nm.msgnums[i] >= nntpstate->current_article_number) && (i > 0) ) {
+                               selected_msgnum = nm.msgnums[i-1];
+                       }
+               }
+               if (selected_msgnum > 0) {
+                       nntpstate->current_article_number = selected_msgnum;
+                       cprintf("223 %ld <FIXME@FIXME>\r\n", nntpstate->current_article_number);
+               }
+               else {
+                       cprintf("422 no previous article in this group 3\r\n");
+               }
+       }
+
+       else if ( (acmd == NNTP_NEXT) && (nm.num_msgs == 0) ) {
+                       cprintf("421 no next article in this group 4\r\n");     // nothing here
+       }
+
+       else if ( (acmd == NNTP_NEXT) && (nntpstate->current_article_number >= nm.msgnums[nm.num_msgs-1]) ) {
+                       cprintf("421 no next article in this group 5\r\n");     // already at the end
+       }
+
+       else if (acmd == NNTP_NEXT) {
+               for (i=0; ((i<nm.num_msgs)&&(selected_msgnum<=0)); ++i) {
+                       if (nm.msgnums[i] > nntpstate->current_article_number) {
+                               selected_msgnum = nm.msgnums[i];
+                       }
+               }
+               if (selected_msgnum > 0) {
+                       nntpstate->current_article_number = selected_msgnum;
+                       cprintf("223 %ld <FIXME@FIXME>\r\n", nntpstate->current_article_number);
+               }
+               else {
+                       cprintf("421 no next article in this group 6\r\n");
+               }
+       }
+
+       // should never get here
+       else {
+               cprintf("500 internal error\r\n");
+       }
+
+       if (nm.msgnums != NULL) {
+               free(nm.msgnums);
+       }
+
+}
+
+
 // 
 // Main command loop for NNTP server sessions.
 //
@@ -893,10 +956,19 @@ void nntp_command_loop(void)
                        || (!strcasecmp(cmdname, "head"))
                        || (!strcasecmp(cmdname, "body"))
                        || (!strcasecmp(cmdname, "stat"))
-               ) {
+               )
+       {
                nntp_article(ChrPtr(Cmd));
        }
 
+       else if (
+                       (!strcasecmp(cmdname, "last"))
+                       || (!strcasecmp(cmdname, "next"))
+               )
+       {
+               nntp_last_next(ChrPtr(Cmd));
+       }
+
        else {
                cprintf("500 I'm afraid I can't do that.\r\n");
        }