Page MenuHomePhorge
Differential D4289

Actually set the host and not the uri
Needs RevisionPublic
Actions

Authored by mollekopf on Apr 26 2023, 6:55 PM.
Tags
None
Referenced Files
F12014568: D4289.diff
Sun, May 5, 5:27 AM
F12010914: D4289.diff
Sun, May 5, 12:49 AM
Unknown Object (File)
Mon, Apr 29, 1:39 PM
Unknown Object (File)
Sun, Apr 28, 9:24 PM
Unknown Object (File)
Sun, Apr 28, 2:56 PM
Unknown Object (File)
Wed, Apr 17, 10:34 AM
Unknown Object (File)
Wed, Apr 17, 10:34 AM
Unknown Object (File)
Mon, Apr 15, 4:00 AM
View All 59 Files
Subscribers
machniak

Details

Reviewers
machniak
Summary

Otherwise ldapsearch get's confused by getting a uri as host and a port
separately.

Diff Detail

Repository
rWAP webadmin
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 42599
Build 17192: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Apr 26 2023, 6:55 PM
mollekopf created this revision.
Harbormaster completed remote builds in B42599: Diff 12326.Apr 26 2023, 6:55 PM
machniak requested changes to this revision.Apr 27 2023, 8:49 AM
machniak subscribed.

I learned recently that something like ldapi:/// exists (https://github.com/roundcube/roundcubemail/issues/8990). I suppose this will break it.

This revision now requires changes to proceed.Apr 27 2023, 8:49 AM

Revision Contents
Changeset List

PathSize
lib/
Auth/
2 lines

Diff 12326

View Options

lib/Auth/LDAP.php

Loading...