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

Include django-auth-ldap in the container image #531

Closed
Exhei opened this issue Aug 9, 2023 · 0 comments · Fixed by #569
Closed

Include django-auth-ldap in the container image #531

Exhei opened this issue Aug 9, 2023 · 0 comments · Fixed by #569
Assignees
@mikedep333
Labels
Feature

Comments

@Exhei
Copy link

Exhei commented Aug 9, 2023

Is your feature request related to a problem? Please describe.
Pulp3 LDAP integration

Describe the solution you'd like
Include django-auth-ldap in the container image
@mikedep333 mikedep333 self-assigned this Aug 15, 2023
git-hyagi added a commit to git-hyagi/pulp-operator that referenced this issue Sep 11, 2023
@git-hyagi
Add support to integrate Pulp with LDAP
7b53f6b 
This PR depends on pulp/pulp-oci-images#531.
There is also a possible issue in the integration between Pulp and
LDAPS/LDAP+TLS using `django-auth-ldap`.
While we don't find a solution, I could workaround the exceptions using:
```
FROM quay.io/pulp/pulp-minimal:3.32
RUN pip3 install django-auth-ldap==4.5.0
RUN sed -i '126i \            if options != None:' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '127i \                options = {int(k):v for k,v in options.items()}' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '859i \                optInt = int(opt)' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '860s/opt, value/optInt, value/' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
```

ref: pulp#518
closes: pulp#1042
git-hyagi added a commit to git-hyagi/pulp-operator that referenced this issue Sep 11, 2023
@git-hyagi
Add support to integrate Pulp with LDAP
e770ea1 
This PR depends on pulp/pulp-oci-images#531.
There is also a possible issue in the integration between Pulp and
LDAPS/LDAP+TLS using `django-auth-ldap`.
While we don't find a solution, I could workaround the exceptions using:
```
FROM quay.io/pulp/pulp-minimal:3.32
RUN pip3 install django-auth-ldap==4.5.0
RUN sed -i '126i \            if options != None:' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '127i \                options = {int(k):v for k,v in options.items()}' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '859i \                optInt = int(opt)' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '860s/opt, value/optInt, value/' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
```

closes: pulp#1042
@git-hyagi git-hyagi mentioned this issue Sep 11, 2023
Merged
git-hyagi added a commit to pulp/pulp-operator that referenced this issue Sep 11, 2023
@git-hyagi
Add support to integrate Pulp with LDAP
9a492ea 
This PR depends on pulp/pulp-oci-images#531.
There is also a possible issue in the integration between Pulp and
LDAPS/LDAP+TLS using `django-auth-ldap`.
While we don't find a solution, I could workaround the exceptions using:
```
FROM quay.io/pulp/pulp-minimal:3.32
RUN pip3 install django-auth-ldap==4.5.0
RUN sed -i '126i \            if options != None:' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '127i \                options = {int(k):v for k,v in options.items()}' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '859i \                optInt = int(opt)' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
RUN sed -i '860s/opt, value/optInt, value/' /usr/local/lib/python3.8/site-packages/django_auth_ldap/backend.py
```

closes: #1042
mikedep333 added a commit to mikedep333/pulp-oci-images that referenced this issue Dec 19, 2023
@mikedep333
Include the pip package django-auth-ldap
eaca2c0 
Needed for Pulp 3 LDAP integration

Fixes: pulp#531
mikedep333 added a commit to mikedep333/pulp-oci-images that referenced this issue Dec 19, 2023
@mikedep333
Include the pip package django-auth-ldap
663bfb6 
Needed for Pulp 3 LDAP integration

Fixes: pulp#531
@mikedep333 mikedep333 mentioned this issue Dec 19, 2023
Merged
mikedep333 added a commit to mikedep333/pulp-oci-images that referenced this issue Dec 19, 2023
@mikedep333
Include the pip package django-auth-ldap
189be5b 
Needed for Pulp 3 LDAP integration

Fixes: pulp#531
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mikedep333 mikedep333

Labels
Feature
Projects
No open projects
RH Pulp Kanban board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Include the pip package django-auth-ldap mikedep333/pulp-oci-images
2 participants
@mikedep333 @Exhei