ObjFW  Diff

Differences From Artifact [0348d66062]:

• File src/OFHTTPClient.m — part of check-in [a61e0594b4] at 2023-10-15 12:32:29 on branch trunk — Add -[OFStream lowlevelIsAtEndOfStream]

  This allows for a much cleaner solution to avoid the internal read
  buffer of e.g. a TLS connection never being processed while waiting for
  a delimiter. (user: js, size: 30941) [annotate] [blame] [check-ins using]

To Artifact [771a949bd6]:

• File src/OFHTTPClient.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: 30943) [annotate] [blame] [check-ins using] [more...]

128
129
130
131
132
133
134
135
136
137
138
139
140
141
142

	if (IRI.path.length > 0)
		path = IRI.percentEncodedPath;
	else
		path = @"/";

	requestString = [OFMutableString stringWithFormat:
	    @"%s %@", OFHTTPRequestMethodName(method), path];

	if (IRI.query != nil) {
		[requestString appendString: @"?"];
		[requestString appendString: IRI.percentEncodedQuery];
	}

	[requestString appendString: @" HTTP/"];

|

128
129
130
131
132
133
134
135
136
137
138
139
140
141
142

	if (IRI.path.length > 0)
		path = IRI.percentEncodedPath;
	else
		path = @"/";

	requestString = [OFMutableString stringWithFormat:
	    @"%@ %@", OFHTTPRequestMethodString(method), path];

	if (IRI.query != nil) {
		[requestString appendString: @"?"];
		[requestString appendString: IRI.percentEncodedQuery];
	}

	[requestString appendString: @" HTTP/"];