Support HTTP basic auth in HTTPFuture
ClosedPublic
Actions

Authored by epriestley on Mar 26 2014, 10:48 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jan 10, 2:59 PM

	Unknown Object (File)
	Sat, Dec 28, 3:33 PM

	Unknown Object (File)
	Dec 5 2024, 11:50 AM

	Unknown Object (File)
	Dec 3 2024, 10:29 AM

	Unknown Object (File)
	Nov 21 2024, 5:29 AM

	Unknown Object (File)
	Nov 20 2024, 10:47 PM

	Unknown Object (File)
	Nov 16 2024, 12:54 AM

	Unknown Object (File)
	Nov 12 2024, 1:06 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan

Maniphest Tasks

T4590: Unable to pass in HTTP auth in the Build Plans->Make HTTP Request section

Commits

rPHU7a4f46251c21: Support HTTP basic auth in HTTPFuture

Summary

Ref T4590. We already sort of support this for Conduit, push the support a little bit deeper.

Test Plan

Added basic auth across my entire nginx. Requests with no auth or with the wrong auth failed; requests with the right auth worked.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20455.Mar 26 2014, 10:48 PM

epriestley retitled this revision from to Support HTTP basic auth in HTTPFuture.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T4590: Unable to pass in HTTP auth in the Build Plans->Make HTTP Request section.

Herald added a subscriber: epriestley.Mar 26 2014, 10:48 PM

btrahan accepted this revision.Mar 26 2014, 10:53 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 26 2014, 10:53 PM

Closed by commit rPHU7a4f46251c21 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

conduit/

ConduitClient.php

15 lines

future/

http/

BaseHTTPFuture.php

19 lines

Diff 20464

View Options

src/conduit/ConduitClient.php