summaryrefslogtreecommitdiff
path: root/lsp-write.c
diff options
context:
space:
mode:
authorpommicket <pommicket@gmail.com>2023-01-11 17:34:27 -0500
committerpommicket <pommicket@gmail.com>2023-01-11 17:34:27 -0500
commitb983ee158ace30f836ec25c4d6a6426281e5700c (patch)
treed326bf2a5ba77d8ac9a6dd47367ef29fc8a7f3e1 /lsp-write.c
parentb0fe15647d196081d801db574fa074ff77e46f5f (diff)
URI %-escape sequences
Diffstat (limited to 'lsp-write.c')
-rw-r--r--lsp-write.c25
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, "\"");
}