search

philbertphotos / osticket-multildap-auth-plugin

Plugin for OS Ticket that allows for authentication with multiple domains.
GNU General Public License v3.0
28 stars 16 forks source link

Uncommented echo #19

Closed tauthement closed 5 years ago

tauthement commented 5 years ago

Hello,

I noticed on line 243 of auth.php that you echo out the SQL statement. This causes the statement to be output on the site. I figured this was unintentional since you commented out other echoes but not this one.

Have a nice day and thanks for your work!

philbertphotos commented 5 years ago

Yea thanks that was for testing and i forgot to remove it ... will update.

Sent from my iPhone

On Nov 9, 2018, at 4:27 PM, Timmy Authement notifications@github.com wrote:

Hello,

I noticed on line 243 of auth.php that you echo out the SQL statement. This causes the statement to be output on the site. I figured this was unintentional since you commented out other echoes but not this one.

Have a nice day and thanks for your work!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

philbertphotos commented 5 years ago

Updated