update the comments in cmd_dlui() to match what I put on the wiki docs

[citadel.git] / citadel / modules / image / serv_image.c

diff --git a/citadel/modules/image/serv_image.c b/citadel/modules/image/serv_image.c
index ff7d2e8e8910650d6675200103c5c10490251a83..3f327324f1fedb876306c677e3075c68e6434170 100644 (file)
--- a/citadel/modules/image/serv_image.c
+++ b/citadel/modules/image/serv_image.c
@@ -42,7 +42,9 @@ void cmd_dlui(char *cmdbuf)
 
        struct CtdlMessage *msg = CtdlFetchMessage(ruser.msgnum_pic, 1, 1);
        if (msg != NULL) {
-               // 600 402132|-1||image/gif|
+               // The call to CtdlOutputPreLoadedMsg() with MT_SPEW_SECTION will cause the DLUI command
+               // to have the same output format as the DLAT command, because it calls the same code.
+               // For example: 600 402132|-1||image/gif|
                safestrncpy(CC->download_desired_section, "1", sizeof CC->download_desired_section);
                CtdlOutputPreLoadedMsg(msg, MT_SPEW_SECTION, HEADERS_NONE, 1, 0, 0);
                CM_Free(msg);
@@ -76,6 +78,7 @@ void import_one_userpic_file(char *username, long usernum, char *path)
                        fread(unencoded_data, data_length, 1, fp);
                        char *encoded_data = malloc((data_length * 2) + 100);
                        if (encoded_data) {
+                               // FIXME try to guess the content-type based on the filename, don't assume GIF
                                sprintf(encoded_data, "Content-type: image/gif\nContent-transfer-encoding: base64\n\n");
                                CtdlEncodeBase64(&encoded_data[strlen(encoded_data)], unencoded_data, data_length, 1);