LSC does not encode/decode characters which are not allowed in File URIs about vim-lsc HOT 5 CLOSED

Curiously in your last example the () are not encoded...

I think I can go with a relatively straightforward encode strategy since we expect the entire file path to be treated as the path component from the URI spec.

1. Allow unreserved characters
2. Allow / unescaped
3. Escape all other characters. Expect that all characters are ascii

My first approach for decoding will probably only handle %nn style encodings. Handling other cases like translating + to space or %% to % could be trickier since it's less general.

from vim-lsc.

Can you try updating to the latest and see if it works for you?

from vim-lsc.

Hello @natebosch ,

Thank you for the update, looks good.
URI encoding for messages like textDocument/definition or textDocument/didOpen appears to be working, but there is one case (I found so far) where URI encoding does not get applied:

{"method":"initialize","jsonrpc":"2.0","id":1,"params":{"rootUri":"file:///Users/tester/projects/Sforce - SFDC Experiments/SForce (lsc-test)/src/classes","capabilities":{...}}

I think this line:

      \ 'rootUri': 'file://'.getcwd(),

should look something like like this:

      \ 'rootUri': lsc#uri#documentUri(getcwd()),

Curiously in your last example the () are not encoded...

that's my bad. Thank you for highlighting this.

from vim-lsc.

Good catch on the rootUri. Should be fixed now.

from vim-lsc.

That's great, thank you.

from vim-lsc.