* Fix '/*' found in comment

[citadel.git] / webcit / context_loop.c

diff --git a/webcit/context_loop.c b/webcit/context_loop.c
index d930fd271fc8ae48bf44cc98869de79372362fd7..3a20c7e82092a54a283457151b3d3862b188fe4f 100644 (file)
--- a/webcit/context_loop.c
+++ b/webcit/context_loop.c
@@ -318,7 +318,13 @@ int ReadHttpSubject(ParsedHttpHdrs *Hdr, StrBuf *Line, StrBuf *Buf)
        if (Hdr->HR.Handler != NULL) {
                if ((Hdr->HR.Handler->Flags & BOGUS) != 0)
                        return 1;
-               Hdr->HR.DontNeedAuth = (Hdr->HR.Handler->Flags & ISSTATIC) != 0;
+               Hdr->HR.DontNeedAuth = (
+                       ((Hdr->HR.Handler->Flags & ISSTATIC) != 0) ||
+                       ((Hdr->HR.Handler->Flags & ANONYMOUS) != 0)
+                       );
+       }
+       else {
+               Hdr->HR.DontNeedAuth = 1; /* Flat request? show him the login screen... */
        }
 
        return 0;
@@ -348,7 +354,7 @@ int AnalyseHeaders(ParsedHttpHdrs *Hdr)
 /*
  * Read in the request
  */
-int ReadHTTPRequset (ParsedHttpHdrs *Hdr)
+int ReadHTTPRequest (ParsedHttpHdrs *Hdr)
 {
        const char *pch, *pchs, *pche;
        OneHttpHeader *pHdr;
@@ -361,7 +367,7 @@ int ReadHTTPRequset (ParsedHttpHdrs *Hdr)
        LastLine = NULL;
        do {
                nLine ++;
-               Line = NewStrBuf();
+               Line = NewStrBufPlain(NULL, SIZ / 4);
 
                if (ClientGetLine(Hdr, Line) < 0) return 1;
 
@@ -427,7 +433,16 @@ int ReadHTTPRequset (ParsedHttpHdrs *Hdr)
        return isbogus;
 }
 
+void OverrideRequest(ParsedHttpHdrs *Hdr, const char *Line, long len)
+{
+       StrBuf *Buf = NewStrBuf();
+
+       FlushStrBuf(Hdr->HR.ReqLine);
+       StrBufPlain(Hdr->HR.ReqLine, Line, len);
+       ReadHttpSubject(Hdr, Hdr->HR.ReqLine, Buf);
 
+       FreeStrBuf(&Buf);
+}
 
 /*
  * handle one request
@@ -454,7 +469,7 @@ void context_loop(ParsedHttpHdrs *Hdr)
        /*
         * Find out what it is that the web browser is asking for
         */
-       isbogus = ReadHTTPRequset(Hdr);
+       isbogus = ReadHTTPRequest(Hdr);
 
        if (!isbogus)
                isbogus = AnalyseHeaders(Hdr);
@@ -492,7 +507,7 @@ void context_loop(ParsedHttpHdrs *Hdr)
                gettimeofday(&tx_finish, NULL);
                
 #ifdef TECH_PREVIEW
-               if ((Hdr->HR.Handler == NULL) ||
+               if ((Hdr->HR.Handler != NULL) ||
                    ((Hdr->HR.Handler->Flags & LOGCHATTY) == 0))
 #endif
                        lprintf(9, "HTTP: 200 [%ld.%06ld] %s %s \n",
@@ -509,9 +524,6 @@ void context_loop(ParsedHttpHdrs *Hdr)
        if (Hdr->HR.got_auth == AUTH_BASIC) 
                CheckAuthBasic(Hdr);
 
-/*
-TODO    HKEY("/static/nocookies.html?force_close_session=yes"));
-*/
 
 /*     dbg_PrintHash(HTTPHeaders, nix, NULL);  */
 
@@ -530,6 +542,12 @@ TODO    HKEY("/static/nocookies.html?force_close_session=yes"));
        if (TheSession == NULL) {
                TheSession = CreateSession(1, &SessionList, Hdr, &SessionListMutex);
 
+               if ((StrLength(Hdr->c_username) == 0) &&
+                   (!Hdr->HR.DontNeedAuth)) {
+                       OverrideRequest(Hdr, HKEY("GET /static/nocookies.html?force_close_session=yes HTTP/1.0"));
+                       Hdr->HR.prohibit_caching = 1;
+               }
+               
                if (StrLength(Hdr->c_language) > 0) {
                        lprintf(9, "Session cookie requests language '%s'\n", ChrPtr(Hdr->c_language));
                        set_selected_language(ChrPtr(Hdr->c_language));
@@ -559,8 +577,10 @@ TODO    HKEY("/static/nocookies.html?force_close_session=yes"));
        gettimeofday(&tx_finish, NULL);
        
 
-       if ((Hdr->HR.Handler == NULL) ||
+#ifdef TECH_PREVIEW
+       if ((Hdr->HR.Handler != NULL) &&
            ((Hdr->HR.Handler->Flags & LOGCHATTY) == 0))
+#endif
                lprintf(9, "HTTP: 200 [%ld.%06ld] %s %s \n",
                        ((tx_finish.tv_sec*1000000 + tx_finish.tv_usec) - (tx_start.tv_sec*1000000 + tx_start.tv_usec)) / 1000000,
                        ((tx_finish.tv_sec*1000000 + tx_finish.tv_usec) - (tx_start.tv_sec*1000000 + tx_start.tv_usec)) % 1000000,
@@ -753,8 +773,8 @@ InitModule_CONTEXT
  * Look for commonly-found probes of malware such as worms, viruses, trojans, and Microsoft Office.
  * Short-circuit these requests so we don't have to send them through the full processing loop.
  */
-       WebcitAddUrlHandler(HKEY("scripts"), do_404, ANONYMOUS|BOGUS); /* /root.exe     /* Worms and trojans and viruses, oh my! */
-       WebcitAddUrlHandler(HKEY("c"), do_404, ANONYMOUS|BOGUS);        /* /winnt */
+       WebcitAddUrlHandler(HKEY("scripts"), do_404, ANONYMOUS|BOGUS);          /* /root.exe - Worms and trojans and viruses, oh my! */
+       WebcitAddUrlHandler(HKEY("c"), do_404, ANONYMOUS|BOGUS);                /* /winnt */
        WebcitAddUrlHandler(HKEY("MSADC"), do_404, ANONYMOUS|BOGUS);
        WebcitAddUrlHandler(HKEY("_vti"), do_404, ANONYMOUS|BOGUS);             /* Broken Microsoft DAV implementation */
        WebcitAddUrlHandler(HKEY("MSOffice"), do_404, ANONYMOUS|BOGUS);         /* Stoopid MSOffice thinks everyone is IIS */
@@ -766,15 +786,14 @@ void
 HttpNewModule_CONTEXT
 (ParsedHttpHdrs *httpreq)
 {
-       httpreq->PlainArgs = NewStrBuf();
-       httpreq->this_page = NewStrBuf();
+       httpreq->PlainArgs = NewStrBufPlain(NULL, SIZ);
+       httpreq->this_page = NewStrBufPlain(NULL, SIZ);
 }
 
 void 
 HttpDetachModule_CONTEXT
 (ParsedHttpHdrs *httpreq)
 {
-       FlushStrBuf(httpreq->ReadBuf);
        FlushStrBuf(httpreq->PlainArgs);
        FlushStrBuf(httpreq->this_page);
        FlushStrBuf(httpreq->PlainArgs);
@@ -787,7 +806,6 @@ HttpDestroyModule_CONTEXT
 (ParsedHttpHdrs *httpreq)
 {
        FreeStrBuf(&httpreq->this_page);
-       FreeStrBuf(&httpreq->ReadBuf);
        FreeStrBuf(&httpreq->PlainArgs);
        FreeStrBuf(&httpreq->this_page);
        FreeStrBuf(&httpreq->PlainArgs);