Error about wrong filename using geticon

Description

Error in journalctl about wrong filename using geticon

Distribution Name and Version

Debian buster

FusionDirectory Version

1.4

PHP version used

7

Origin of php packages

Debian

Steps to Reproduce

Play a bit with FustionDirectory (connect, create user, disconnect) and look the logs

Expected behavior:

No error

Actual behavior:

There is an error in the logs

Reproduces how often: 100%

Additional Information

Error message

 fév 19 14:24:46 buster php[2569]: FusionDirectory [unauthenticated]: (error) /usr/share/fusiondirectory/html/geticon.php of type php : Type:2, Message:readfile(): Filename cannot be empty, File:/usr/share/fusiondirectory/html/geticon.php, Line: 51

changed milestone to %FusionDirectory 1.4

By Jonathan Swaelens on 2020-02-19T13:53:54 (imported from GitLab)

added Bugs PJ1802-0188 fusiondirectory-core labels

added 5m of time spent at 2020-02-19

By Jonathan Swaelens on 2020-02-19T13:54:05 (imported from GitLab)

created merge request !742 to address this issue

By Côme Chilliet on 2020-02-19T14:13:28 (imported from GitLab)

mentioned in merge request !742

By Côme Chilliet on 2020-02-19T14:13:28 (imported from GitLab)

mentioned in commit bf2dfa7b

By Côme Chilliet on 2020-02-19T14:19:25 (imported from GitLab)

@jswaelens You should now get the query string in the logs when an icon is not found

By Côme Chilliet on 2020-02-19T14:49:29 (imported from GitLab)

added To Be Tested label

assigned to @jswaelens and unassigned @MCMic

By Côme Chilliet on 2020-02-19T14:49:30 (imported from GitLab)

Hello @MCMic

It look like it's related to webauthn that not have icon

fév 19 16:40:47 buster php[4796]: FusionDirectory [unauthenticated]: (error) /usr/share/fusiondirectory/html/geticon.php of type php : Type:1024, Message:Could not find icon for context=applications&icon=webauthn&size=48, File:/usr/share/fusiondirectory/html/geticon.php, Line: 44

By Jonathan Swaelens on 2020-02-19T15:42:44 (imported from GitLab)

I close this issue I will add a comment about the webauthn icon in the webauthn plugin issue

By Jonathan Swaelens on 2020-02-19T15:51:09 (imported from GitLab)

closed

By Jonathan Swaelens on 2020-02-19T15:51:09 (imported from GitLab)

removed To Be Tested label

mentioned in issue fd-plugins#6013 (closed)

By Jonathan Swaelens on 2020-02-19T15:55:09 (imported from GitLab)

removed Bugs label

added Fixed label