Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

WIP - Projects, Users, and Auth(z|n) #86

Closed
wants to merge 7 commits into from
Closed

WIP - Projects, Users, and Auth(z|n) #86

wants to merge 7 commits into from

Conversation

smarterclayton
Copy link
Contributor

No description provided.

@smarterclayton
Copy link
Contributor Author

@erictune Eric, I'm using this pull as a basis for authn / authz prototyping (I'm still plumbing things below the covers with users, auth, oath, etc). Once I get the basic stuff wired up, I'm going to take a stab at defining filters and composable tools for wrapping a deployment in proxy layers. Any updates on the policy side?

@erictune
Copy link

Hoping to do a PR by end-of-day tomorrow with design for authorization
policy, for your consideration.

On Mon, Sep 15, 2014 at 11:43 AM, Clayton Coleman notifications@github.com
wrote:

@erictune https://github.com/erictune Eric, I'm using this pull as a
basis for authn / authz prototyping (I'm still plumbing things below the
covers with users, auth, oath, etc). Once I get the basic stuff wired up,
I'm going to take a stab at defining filters and composable tools for
wrapping a deployment in proxy layers. Any updates on the policy side?

—
Reply to this email directly or view it on GitHub
#86 (comment).

@openshift-bot
Copy link
Contributor

Origin Action Required: Pull request cannot be automatically merged, please rebase your branch from latest HEAD and push again

@erictune erictune mentioned this pull request Sep 29, 2014
Closed
@smarterclayton
Copy link
Contributor Author

Superseded by #218

ironcladlou pushed a commit to ironcladlou/origin that referenced this pull request Oct 15, 2014
@pmorie
Merge pull request openshift#86 from pmorie/deploy-trigger-wip
49c8eca 
Refactor config change controller and tests
@smarterclayton smarterclayton deleted the projects branch May 19, 2015 02:40
@sspeiche sspeiche mentioned this pull request Sep 3, 2015
Closed
deads2k referenced this pull request in deads2k/origin May 17, 2016
@sdodson
Merge pull request #86 from sdodson/origin24efed3f
9e01297 
Origin24efed3f
danwinship pushed a commit to danwinship/origin that referenced this pull request Jun 24, 2016
Merge pull request openshift#86 from rajatchopra/master
e426497 
install docker systemd conf file
jpeeler pushed a commit to jpeeler/origin that referenced this pull request Feb 1, 2018
@bmelville @pmorie
Extract broker communication code to a client. (openshift#86)
6b30011 
* Extract broker communication code to a client.

This introduces a broker client interface and extracts the communcation
code from the controller into a CF Service Broker V2 API client.

Signed-off-by: Brendan Melville <bmelville@google.com>

* Address comments around HTTP in broker client.

Signed-off-by: Brendan Melville <bmelville@google.com>

* Add comments for all BrokerClient funcs.

Also adds some error handling from PR comments.

Signed-off-by: Brendan Melville <bmelville@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@smarterclayton @erictune @openshift-bot