mattv8 / yourls-ldap-plugin

Simple LDAP plugin for YOURLS

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Miss spelling at LDAPAUTH variables.

surauki opened this issue · comments

It's so tiny issues. I'm sorry...
I found miss spelling in plugin.php when setting port variables.

  • plugin.php
function ldapauth_environment_check() {
	$required_params = array(
		'LDAPAUTH_HOST', // ldap host
		//'LDAAUTHP_PORT', // ldap port
		'LDAPAUTH_BASE', // base ldap path
		//'LDAPAUTH_USERNAME_FIELD', // field to check the username against
	);
  • Incorrect
    //'LDAAUTHP_PORT', // ldap port

  • Correct
    //'LDAPAUTH_PORT', // ldap port

that all. thank you developing !

commented

Minor but why not fix it. Thanks for reporting! :)