Curtis Conard comments

Results 374 comments of


                                            Curtis Conard

Fix LDAP DC case sensitivity

I also tried making tests (last week). The LDAP server does not recognize the memberof attribute and apparently there is an overlay/module needed to add support but I've had no...

Test Notification is sending mails FROM administrator mail (should use the sender mail)

Fixed by #12291 and confirmed.

[10.0.0] Ticket::check() rights for UPDATE not called

Do you have an example of a plugin that has this issue?

[10.0.0] Ticket::check() rights for UPDATE not called

The right check is still done via the canUpdate function. The difference is that it checks UPDATE, ASSIGN, STEAL, OWN, and CHANGEPRIORITY rights and if the user has any, the...

Preserve tags case when text notification is built from html

This would remove the conversion of any bold text to uppercase if the preserve_case argument is false even if it is not a tag and I don't know if that...

Error 512 pending reason

Can you provide steps to recreate the issue?

Before you add the task, is there a pending reason template selected and are the pending reason options populated properly? ![image](https://user-images.githubusercontent.com/17678637/182990908-84492b2c-560c-442d-9e63-f691b0574309.png)

API: decode html

I don't agree with decoding every field in the response, at least not now given that it has always been encoded. What if they were passing the encoded values as-is...

API: decode html

Another possibility is a header that lets the client specify which fields they want to be unsanitized like: `X-Glpi-Unsanitized-Fields: name, content`

API: decode html

I cannot test this at the moment. The API authenticates me but doesn't load my profiles or entities into the session. I do not think it is related to this...