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

	F18985485: D21251.id50606.diff
	Mon, Nov 17, 1:04 PM

	F18875875: D21251.id50606.diff
	Wed, Nov 5, 7:57 PM

	F18870086: D21251.diff
	Tue, Nov 4, 3:22 PM

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

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

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

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

	F18404171: D21251.diff
	Aug 29 2025, 9:57 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...