Skip to content
This repository has been archived by the owner on Apr 11, 2024. It is now read-only.

Fix ldaps connect #34

Merged
merged 1 commit into from
Nov 30, 2022
Merged

Fix ldaps connect #34

merged 1 commit into from
Nov 30, 2022

Conversation

Alexconquer
Copy link
Contributor

Fix for ldaps error #24

@hydrian
Copy link
Owner

hydrian commented Jan 18, 2019

What version of php did you test your code on?

@Alexconquer
Copy link
Contributor Author

I use PHP 7.0.33-0+deb9u1

@kettbi kettbi mentioned this pull request Mar 13, 2019
@faust64
Copy link

faust64 commented Oct 24, 2020

Using the original plugin, I can see tinytinyrss failing to connect my OpenLDAP, with the following message in slapd logs:

5f943ab4 conn=7283 fd=16 ACCEPT from IP=x.x.x.x:59934 (IP=0.0.0.0:1636)
TLS: can't accept: error:1408F10B:SSL routines:ssl3_get_record:wrong version number.

In tt-rss logs:

IP: X.X.X.X. Request URI: /tt-rss/public.php?return=https%3A%2F%2Ftinytinyrss.x.x.x.com%2Ftt-rss%2FUser agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.75 Safari/537.36 |   | 16:27
-- | -- | --
E_USER_NOTICE (1024) | :0 | Array ( [host] => openldap-kube.ci.svc.cluster.local [basedn] => dc=xxx,dc=com [port] => 1636 [starttls] => )

IP: X.X.X.X Request URI: /tt-rss/public.php?return=https%3A%2F%2Ftinytinyrss.x.x.x.com%2Ftt-rss%2FUser agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.75 Safari/537.36 |   | 16:28
-- | -- | --
E_USER_ERROR (256) | :0 | LDAP bind(): Bind failed ()with DN cn=tinytinyrss,ou=services,dc=xxx,dc=com

I confirm applying that patch fixed it. Thanks @Alexconquer

Using PHP 7.4. How wasn't this merged already?

@hydrian hydrian merged commit 582ade4 into hydrian:master Nov 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants