#45 Query content_set from Pulp
Merged by cqi. Opened by cqi.
cqi/freshmaker query-pulp  into  master

Download 45.patch
no initial comment

Is authentication mandatory with Pulp?

I left one question, but in general this looks good and straightforward to me.

If authentication is not mandatory, let's drop that (to keep things simpler). Otherwise, :+1: from me.

rebased

Rebased.

@ralph AFAIK, internal pulp is just using Basic authentication. It's easy to support Kerberos authentication by using requests. I'm happy to do that if it is needed.

Rebased.
@ralph AFAIK, internal pulp is just using Basic authentication. It's easy to support Kerberos authentication by using requests. I'm happy to do that if it is needed.

I think the point is whether authentication is required for querying content_set, if not, it's unnecessary to have authentication in this api.

I think the point is whether authentication is required for querying content_set, if not, it's unnecessary to have authentication in this api.

ok, maybe need a ping to @sochotni

Thanks @qwan for clarifying.

Yes, basic auth is required for all queries

rebased

Pull-Request has been merged by cqi

Metadata