Differences From Artifact [df8e40fa8b]:
- File src/OFHTTPServer.m — part of check-in [fc161d1948] at 2023-05-21 11:04:32 on branch trunk — OFTCPSocket: Return socket address bound to (user: js, size: 20125) [annotate] [blame] [check-ins using] [more...]
To Artifact [c229043b21]:
- File
src/OFHTTPServer.m
— part of check-in
[12c09ef41e]
at
2023-10-15 14:55:50
on branch trunk
— Add OFHTTPRequestMethodString()
This deprecates OFHTTPRequestMethodName(), which returns a C string.
APIs should avoid C strings as much as possible.This function was initially only used internally, where this was fine.
However, when it was made public, it should have been converted to
OFString at the same time.Adds OFHTTPRequestMethodParseString() for consistency, which behaves the
same as OFHTTPRequestMethodParseName() and deprecates it. (user: js, size: 20104) [annotate] [blame] [check-ins using] [more...]
| ︙ | ︙ | |||
378 379 380 381 382 383 384 |
pos = [line rangeOfString: @" "].location;
if (pos == OFNotFound)
return [self sendErrorAndClose: 400];
method = [line substringToIndex: pos];
@try {
| | | 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 |
pos = [line rangeOfString: @" "].location;
if (pos == OFNotFound)
return [self sendErrorAndClose: 400];
method = [line substringToIndex: pos];
@try {
_method = OFHTTPRequestMethodParseString(method);
} @catch (OFInvalidArgumentException *e) {
return [self sendErrorAndClose: 405];
}
@try {
OFRange range = OFMakeRange(pos + 1, line.length - pos - 10);
|
| ︙ | ︙ | |||
731 732 733 734 735 736 737 | objc_autoreleasePoolPop(pool); return 0; } } | | | | 731 732 733 734 735 736 737 738 739 740 741 742 743 744 745 746 747 |
objc_autoreleasePoolPop(pool);
return 0;
}
}
- (bool)lowlevelHasDataInReadBuffer
{
return _socket.hasDataInReadBuffer;
}
- (int)fileDescriptorForReading
{
return _socket.fileDescriptorForReading;
}
|
| ︙ | ︙ |