Differential D22349

lspclient: Don't send the Content-Type header
ClosedPublic
Actions

Authored by ogoffart on Jul 9 2019, 5:32 PM.

Details

Reviewers
cullmann
mnauwelaerts
Commits
R40:752d40b4e27b: lspclient: Don't send the Content-Type header
Summary

This is the default anyway.
There is a bug in the rust LSP server making it reject this header

Test Plan

Tested with the rust LSP server

Diff Detail

Repository
R40 Kate
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ogoffart created this revision.Jul 9 2019, 5:32 PM
Restricted Application added a project: Kate. · View Herald TranscriptJul 9 2019, 5:32 PM
Restricted Application added a subscriber: kwrite-devel. · View Herald Transcript
ogoffart requested review of this revision.Jul 9 2019, 5:32 PM
cullmann added a comment.Jul 10 2019, 7:03 AM

We pinged the lspclient author to look at these requests (and create an account here, we didn't find the right one to add ;=)

mnauwelaerts accepted this revision.Jul 10 2019, 4:55 PM
mnauwelaerts added a reviewer: mnauwelaerts.
mnauwelaerts accepted this revision.Jul 10 2019, 5:53 PM
This revision is now accepted and ready to land.Jul 10 2019, 5:53 PM
Closed by commit R40:752d40b4e27b: lspclient: Don't send the Content-Type header (authored by ogoffart). · Explain WhyJul 11 2019, 5:26 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 61561

View Options

addons/lspclient/lspclientserver.cpp

Loading...