Page MenuHomePhabricator
Differential D16495

Add Slack authentication adapter
ClosedPublic
Actions

Authored by chad on Sep 6 2016, 5:25 AM.
Tags
None
Referenced Files
F18833782: D16495.id39704.diff
Oct 26 2025, 5:46 AM
F18805123: D16495.id39699.diff
Oct 18 2025, 1:55 PM
F18757438: D16495.id39706.diff
Oct 5 2025, 5:27 PM
F18624060: D16495.id39704.diff
Sep 15 2025, 6:35 PM
F18622040: D16495.id39706.diff
Sep 15 2025, 10:15 AM
F18596426: D16495.diff
Sep 13 2025, 12:40 AM
F18089643: D16495.id39699.diff
Aug 6 2025, 11:55 AM
Unknown Object (File)
May 28 2025, 4:57 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rPHUb00eff539ee7: Add Slack authentication adapter
Summary

This adds an authentication adapter to libphutil for Slack. Diff 1 of 2.

Test Plan

Create a new auth provider for Slack. Link Slack account to phacility.slack.com. Authenticate a new user, create an account with Slack reg provider.

Diff Detail

Repository
rPHU libphutil
Branch
slack-oauth
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13583
Build 17496: Run Core Tests
Build 17495: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 39699.Sep 6 2016, 5:25 AM
chad retitled this revision from to Add Slack authentication adapter.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptSep 6 2016, 5:25 AM
epriestley accepted this revision.Sep 6 2016, 1:47 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/future/slack/PhutilSlackFuture.php
51

This should probably be ltrim($this->action, '/') to strip leading slashes, or maybe just omit it entirely.

This revision is now accepted and ready to land.Sep 6 2016, 1:47 PM
chad updated this revision to Diff 39704.Sep 6 2016, 3:50 PM
chad edited edge metadata.
  • per comments
Closed by commit rPHUb00eff539ee7: Add Slack authentication adapter (authored by chad, committed by chad). · Explain WhySep 6 2016, 3:51 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
4 lines
auth/
61 lines
future/
slack/
87 lines

Diff 39704

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/auth/PhutilSlackAuthAdapter.php

Loading...
View Options

src/future/slack/PhutilSlackFuture.php

Loading...
Phabricator · Built by Phacility