Comments (4)
synopse
commented on May 25, 2024
This is weird, because the length comes directly from the Windows API, i.e. Req^.CookedUrl.FullUrlLength.
Please try https://synopse.info/fossil/info/cb30095a75
from mormot.
chrswgnr
commented on May 25, 2024
I tested your patch, with those changes it's fixed.
Well maybe a bug in the windows api then. I am on a Windows 10 Enterprise 1909 right now.
from mormot.
synopse
commented on May 25, 2024
After more checking, the length was in bytes, not WideChars (as would be logical), so my code was incorrect.
Using ending #0 as in my fix is always good, and not really slower, so we will keep it. ;)
Thanks for the report.
from mormot.
chrswgnr
commented on May 25, 2024
Alright! Thanks for the quick reply and fix! :)
from mormot.
Related Issues (20)
- Problem in SynCrtSock.TWinHTTP.InternalSendRequest if the server asks for a client certificate HOT 2
- I have modified/corrected some code but I can't share my work... HOT 3
- Correct criticalsection managment and move some declaration to handle HttpAPI in Windows. HOT 2
- Memory Leak on RemoteDB HOT 2
- Documentation example mistake ? 4.3.1. TList-like properties HOT 10
- Endless recursive calls in SynCommons::FileSize HOT 3
- Typo in SAD 1.18 HOT 3
- why cannot run the same SQLite on Linux at the same time HOT 1
- ARM9 without FPU(VFP) does not support HOT 1
- Forward slash instead of backslash? HOT 3
- Fatal Error (Exception External:?) after second exception in Interfaced based Services HOT 1
- Declaration of "ConvertToEmfPlus" is not 64 bit safe HOT 1
- THttpApiWebSocketConnection.PrivateData Property value cannot be set. HOT 2
- THttpServer.OnHttpRequest - Ctxt.RemoteIP is empty HOT 2
- HttpGet: outStatus parameter is invalid
- THttpServer : Executing the HEAD request will read the entire file HOT 4
- TWinHTTP Thread safe issue in filling OSVersionInfo HOT 1
- Postgresql boolean (t/f) JSON parsing HOT 1
- Cannot compile under Delphi 12 due to the compiler errors HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from mormot.