Paths

Table of Contentst

Differential D21251

Add "HTTPSFuture->addCurlOption()" for raw access to "curl_setopt()"
ClosedPublic
Actions

Authored by epriestley on May 14 2020, 4:14 PM.

Tags

None

Referenced Files

	F18826980: D21251.id50606.diff
	Fri, Oct 24, 6:59 AM

	F18806039: D21251.diff
	Sat, Oct 18, 5:29 PM

	F18790818: D21251.id50607.diff
	Wed, Oct 15, 7:32 PM

	F18734294: D21251.id.diff
	Tue, Sep 30, 10:51 PM

	F18404171: D21251.diff
	Aug 29 2025, 9:57 PM

	F18101109: D21251.diff
	Aug 9 2025, 12:19 PM

	F18083220: D21251.id50606.diff
	Aug 5 2025, 9:24 AM

	F17948732: D21251.diff
	Jul 31 2025, 9:55 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13533: Support arbitrary cURL options, for Kerberos auth
Commits: rARCb76b9c4065bb: Add "HTTPSFuture->addCurlOption()" for raw access to "curl_setopt()"

Summary

Fixes T13533. This is a narrow, fragile API for a particular Kerberos use case on one install.

Test Plan

Set a non-scalar key, got an exception.
Set <"duck", "quack">, got an exception from cURL that the value was invalid.
Set a bunch of made-up options to arbitrary values, no errors. cURL accepts anything so there's nothing we can do about this.
Set CURLOPT_NOBODY and saw the request behavior change, demonstrating that the call can produce effects.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 14 2020, 4:14 PM

Harbormaster completed remote builds in B24423: Diff 50606.May 14 2020, 4:15 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 14 2020, 4:15 PM

epriestley requested review of this revision.

Closed by commit rARCb76b9c4065bb: Add "HTTPSFuture->addCurlOption()" for raw access to "curl_setopt()" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCb76b9c4065bb: Add "HTTPSFuture->addCurlOption()" for raw access to "curl_setopt()".

Revision Contents
Changeset List

Path

Size

src/

future/

http/

HTTPSFuture.php

36 lines

Diff 50607

src/future/http/HTTPSFuture.php

Loading...