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

	Unknown Object (File)
	Wed, Nov 29, 7:35 PM

	Unknown Object (File)
	Sat, Nov 25, 8:54 AM

	Unknown Object (File)
	Fri, Nov 24, 2:40 AM

	Unknown Object (File)
	Sun, Nov 12, 9:24 AM

	Unknown Object (File)
	Wed, Nov 8, 5:50 PM

	Unknown Object (File)
	Wed, Nov 8, 5:38 PM

	Unknown Object (File)
	Wed, Nov 8, 2:21 PM

	Unknown Object (File)
	Wed, Nov 8, 4:31 AM

View All 31 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

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

Path

Size

lib/

Auth/

LDAP.php

2 lines

Diff 12326

View Options

Actually set the host and not the uriNeeds RevisionPublicActions