Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Images Don't Handle Uppercase #205

Open
vinaykapadia opened this issue May 17, 2019 · 3 comments
Open

Images Don't Handle Uppercase #205

vinaykapadia opened this issue May 17, 2019 · 3 comments
Assignees

Comments

@vinaykapadia
Copy link

Your environment:

  • Operating system: Docker on Windows, Linux Container monitorr/monitorr

  • Webserver: NGINX 1.12.2

  • PHP Version: 7.1.17

  • Monitorr Version: 1.7.6m

Describe your issue:

Looks like the images don't handle uppercase. There is a built-in image qBittorrent.png. I added a qBittorrent service and used that image. When I go to the homepage, it shows a broken image. Inspecting that image shows "qbittorrent.png". If I change it to "qBittorrent.png" it shows up. The services_settings-data.json file has it correct, "qBittorrent.png" but when it gets sent to the browser, it gets sent as "qbittorrent.png".

I have read the t-shooting Wiki at: https://github.com/Monitorr/Monitorr/wiki/06-Troubleshooting

@seanvree seanvree self-assigned this May 31, 2019
@seanvree
Copy link
Member

seanvree commented May 31, 2019

@vinaykapadia This is interesting and I thought we had addressed it a while back in an update.

What browser are you using? I've tested on Chrome and Firefox (on Windows 10) and the default image is displayed despite the case of the filename in the services_settings-data.json file or the actual filename on the web server.

image

Let me know.

@angrycuban13
Copy link

I was actually having the same issue. Running it on Docker on latest dev. I uploaded the qBit file myself in order to get around the issue. Browser was Firefox in my case.

@seanvree
Copy link
Member

Yup, you guys are right. Stupid Docker.

THX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants