ilya-biryukov added inline comments.
================ Comment at: clangd/JSONRPCDispatcher.cpp:167 + // It's another header, ignore it. continue; + } else { ---------------- puremourning wrote: > ilya-biryukov wrote: > > Maybe log the skipped header? > I think this would just be noise for conforming clients which send the > (legitimate) Content-Type header. We could of course special case this, but > I'm not sure the extra logging would be all that useful. You're right. Logging `Content-Type` headers doesn't make any sense. https://reviews.llvm.org/D37282 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits