diff options
author | pommicket <pommicket@gmail.com> | 2023-01-11 17:34:27 -0500 |
---|---|---|
committer | pommicket <pommicket@gmail.com> | 2023-01-11 17:34:27 -0500 |
commit | b983ee158ace30f836ec25c4d6a6426281e5700c (patch) | |
tree | d326bf2a5ba77d8ac9a6dd47367ef29fc8a7f3e1 /lsp-write.c | |
parent | b0fe15647d196081d801db574fa074ff77e46f5f (diff) |
URI %-escape sequences
Diffstat (limited to 'lsp-write.c')
-rw-r--r-- | lsp-write.c | 25 |
1 files changed, 24 insertions, 1 deletions
diff --git a/lsp-write.c b/lsp-write.c index 8472190..19bd766 100644 --- a/lsp-write.c +++ b/lsp-write.c @@ -188,7 +188,30 @@ static void write_file_uri(JSONWriter *o, LSPDocumentID document) { // why the fuck is there another slash it makes no goddamn sense str_builder_append(&o->builder, "/"); #endif - write_escaped(o, path); + for (const char *p = path; *p; ++p) { + char c = *p; + #if _WIN32 + // i think file URIs have to use slashes? + if (c == '\\') c = '/'; + #endif + + // see https://www.rfc-editor.org/rfc/rfc3986#page-12 + // these are the only allowed un-escaped characters in URIs + bool escaped = !( + (c >= '0' && c <= '9') + || (c >= 'a' && c <= 'z') + || (c >= 'A' && c <= 'Z') + || c == '_' || c == '-' || c == '.' || c == '~' || c == '/' + #if _WIN32 + || c == ':' // i dont think you're supposed to escape the : in C:\... + #endif + ); + if (escaped) { + str_builder_appendf(&o->builder, "%%%02x", (uint8_t)c); + } else { + str_builder_appendf(&o->builder, "%c", c); + } + } str_builder_append(&o->builder, "\""); } |