starred/pocket-id
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2025-12-06 09:13:19 +03:00
Code Issues 121 Packages Projects Releases 16 Wiki Activity

🐛 Bug Report: broken logo in email #423

New Issue
Closed
opened 2025-10-07 00:14:28 +03:00 by OVERLORD · 4 comments
OVERLORD commented 2025-10-07 00:14:28 +03:00
Owner
Copy Link

Originally created by @cirrusflyer on GitHub.

Reproduction steps

When email is sent from Pocket ID the logo is broken. Looks like a misspelling in the code. It's being sent as:

...api/application-configuratio=n/logo

And should be:

...api/application-configuration/logo

Expected behavior

The logo should display.

Actual Behavior

The logo is broken.

Originally created by @cirrusflyer on GitHub. ### Reproduction steps When email is sent from Pocket ID the logo is broken. Looks like a misspelling in the code. It's being sent as: ...api/application-configuratio=n/logo And should be: ...api/application-configuration/logo ### Expected behavior The logo should display. ### Actual Behavior The logo is broken.
OVERLORD added the bug label 2025-10-07 00:14:28 +03:00
OVERLORD commented 2025-10-07 00:14:30 +03:00
Author
Owner
Copy Link

@EweSparky commented on GitHub:

I'm using the latest as of today. Using Fastmail web client but just confirmed it does display correctly in other email clients. Very odd. Here's what I see in the raw message:

<img src=3D"https://mydomain.com/api/application-configuratio=

n/logo" alt=3D"Pocket ID"/>

I had the same issue; Fastmail said it's because they proxy all images, but they can't do it if they can't reach your Pocket ID instance.

@EweSparky commented on GitHub: > I'm using the latest as of today. Using Fastmail web client but just confirmed it does display correctly in other email clients. Very odd. Here's what I see in the raw message: > > > > <img src=3D"https://mydomain.com/api/application-configuratio= > > n/logo" alt=3D"Pocket ID"/> > > I had the same issue; Fastmail said it's because they proxy all images, but they can't do it if they can't reach your Pocket ID instance.
OVERLORD commented 2025-10-07 00:14:30 +03:00
Author
Owner
Copy Link

@stonith404 commented on GitHub:

Strange, in can't see a typo in the code and it's working for me.

Could you share a bit more information?

  • Which version of Pocket ID are you using?
  • Which email client are you using?
  • Does it work in another email client?
  • Do you use a custom logo?
  • Do you maybe use a SMTP server that alters the email content?
@stonith404 commented on GitHub: Strange, in can't see a typo in the [code](https://github.com/stonith404/pocket-id/blob/204313aacf901b804e86a286cbcdf10e20b47404/backend/internal/service/email_service.go#L69) and it's working for me. Could you share a bit more information? - Which version of Pocket ID are you using? - Which email client are you using? - Does it work in another email client? - Do you use a custom logo? - Do you maybe use a SMTP server that alters the email content?
OVERLORD commented 2025-10-07 00:14:31 +03:00
Author
Owner
Copy Link

@cirrusflyer commented on GitHub:

I'm using the latest as of today. Using Fastmail web client but just confirmed it does display correctly in other email clients. Very odd. Here's what I see in the raw message:

<img src=3D"https://mydomain.com/api/application-configuratio=
n/logo" alt=3D"Pocket ID"/>

@cirrusflyer commented on GitHub: I'm using the latest as of today. Using Fastmail web client but just confirmed it does display correctly in other email clients. Very odd. Here's what I see in the raw message: <img src=3D"https://mydomain.com/api/application-configuratio= n/logo" alt=3D"Pocket ID"/>
OVERLORD commented 2025-10-07 00:14:31 +03:00
Author
Owner
Copy Link

@stonith404 commented on GitHub:

That’s strange. It seems that Fastmail might be interpreting /n as a new line, or another rendering issue on their end. I would recommend reaching out to Fastmail to report this bug. Implementing a workaround in the project for what appears to be a Fastmail-specific issue doesn’t seem like the right approach to me.

Let me know if you need any more information for reporting this to Fastmail.

@stonith404 commented on GitHub: That’s strange. It seems that Fastmail might be interpreting `/n` as a new line, or another rendering issue on their end. I would recommend reaching out to Fastmail to report this bug. Implementing a workaround in the project for what appears to be a Fastmail-specific issue doesn’t seem like the right approach to me. Let me know if you need any more information for reporting this to Fastmail.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
documentation
feature
needs more upvotes
open to pull requests
pull-request
Mirrored from GitHub Pull Request
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id#423
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?