graphql-platform icon indicating copy to clipboard operation
graphql-platform copied to clipboard

Published 20 hours ago • verified publisher icon ChilliCream

Opa Authorization Handler for AspNetCore

Open queil opened this issue 2 years ago • 4 comments

Implement authorization handler delegating authz to an OPA (Open Policy Agent) REST API endpoint. The handler should send a POST request to an OPA Data API endpoint.

queil avatar Jun 13 '22 13:06 queil

@PascalSenn I think I am ready for a review.

queil avatar Jun 14 '22 19:06 queil

Hey sorry for the wait ... we will do the review this week.

michaelstaib avatar Jul 06 '22 15:07 michaelstaib

I will do a bit of reformatting so that it fits with the code-style. Looks great so far.

michaelstaib avatar Jul 12 '22 06:07 michaelstaib

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Aug 17 '22 14:08 CLAassistant

I have merged it now into the V13 branch aka main. I will go over the interfaces before we do a final release. I think a couple could be internal .... I will ship the new package with preview 74

michaelstaib avatar Oct 30 '22 16:10 michaelstaib

Hey @queil in the test is a reference to a local server and all tests seem always to fail. Can you containerize this server? So we can package it with squadron for the tests?

michaelstaib avatar Nov 04 '22 20:11 michaelstaib

Sure, it should be trivial. I'll take a look shortly.

queil avatar Nov 05 '22 01:11 queil

Awesome thank you

michaelstaib avatar Nov 05 '22 20:11 michaelstaib

OPA Server is already available as a Docker image. Does this work for you guys? https://hub.docker.com/r/openpolicyagent/opa/ Also, I'd probably should remove the embedded server from the tests though.

queil avatar Nov 08 '22 15:11 queil

Yes, lets package that up in squadron and then we can use the docker image in the tests.

michaelstaib avatar Nov 10 '22 19:11 michaelstaib

I will create an issue

michaelstaib avatar Nov 10 '22 19:11 michaelstaib