Paul Scheduikat
Paul Scheduikat
I have ~450 notes and it's running on a reasonably fast server with nextcloud data on a ssd (30ms ping time from my phone). Here is a screen recording (switched...
Attached a debugger to a build of current master, some observations: Both etag and modified are always "0". `onlyLocalChanges` is always false All 450 Notes are downloaded in full on...
@stefan-niedermann I fixed this in my PR, did it truly work for you before? If so I might need to check the headers without taking capitalization into account.
My response headers are as follows: ``` "x-notes-api-versions" -> "0.2, 1.3" "Server" -> "cloudflare" "vary" -> "accept-encoding" "x-frame-options" -> "SAMEORIGIN" "last-modified" -> "Mon, 30 Dec 2024 20:52:52 GMT" "Report-To" ->...
Header names are not case sensitive. From [RFC 2616 - "Hypertext Transfer Protocol -- HTTP/1.1"](http://www.w3.org/Protocols/rfc2616/rfc2616.html), [Section 4.2, "Message Headers"](http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2): Each header field consists of a name followed by a colon...
@stefan-niedermann Opened a PR upstream as I think it's cleaner to fix it there.
We need some code to reproduce this
Thanks for the investigation, however it's not our code which generates the replacement for the strings that's slow it's a regression in the go compiler. Hopefully it's fixed in 1.25,...
@mvdan If you have the time, maybe time if we or go is the problem
@JustinTimperio Can you check with go 1.25.0 and https://github.com/burrowers/garble/pull/970 ? Still has some issues but should work for a quick timing test.