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

Broken avatar on GHE (again) #1224

Open
1 task done
LunaticMuch opened this issue Jun 12, 2024 · 27 comments
Open
1 task done

Broken avatar on GHE (again) #1224

LunaticMuch opened this issue Jun 12, 2024 · 27 comments
Assignees
Labels
bug Something isn't working host:github-server Related to GitHub Enterprise Server priority:low Items of low importance. Applicable to select few users or use-cases

Comments

@LunaticMuch
Copy link

🔍 Is there already an issue for your problem?

  • I have checked older issues, open and closed

📝 Description

In a GHE situation, avatars for users are broken.
This seems similar to #564 but in fact is not.

Looking at the inspector tab, the src for the image is correctly populated and points a valid image if I open it from my browser.
Incidentally, I have an amount of warnings as

reading cookie in cross-site context will be blocked in future Chrome versions

Whether that's the real problem or not...I am yet to understand. I cannot find any other error.

🪜 Steps To Reproduce

  1. Use in a GHE context
  2. GHE is 3.11

Gitify Version

5.7.0

Operating System

macOS

GitHub Account

GitHub Enterprise

📸 Screenshots

Screenshot 2024-06-12 at 10 39 10

@LunaticMuch LunaticMuch added the bug Something isn't working label Jun 12, 2024
@setchy
Copy link
Member

setchy commented Jun 12, 2024

thanks for reporting @LunaticMuch

so to confirm, the avatar url returns a valid image, but renders as broken 🤔

@LunaticMuch
Copy link
Author

so to confirm, the avatar url returns a valid image, but renders as broken 🤔

Yes. I checked the code, the console, but I can get my head around it. I only suspected the cookie, but also I cannot see the "relationship"

@setchy
Copy link
Member

setchy commented Jun 12, 2024

Puzzling indeed... together I'm sure we'll get to the bottom of it 🤝

Any chance you are able to run gitify from the source code to see if that has the same issue? 🤞

git clone
pnpm i
pnpm watch
pnpm start

@bmulholland
Copy link
Collaborator

I also wonder whether there's a network request for the image at all? Maybe there's a security policy that prevents the loading, for instance. Would need @LunaticMuch to check that, I don't have GHE to try to repro

@LunaticMuch
Copy link
Author

@setchy and @bmulholland I can definitely help... give me a bit, feels more a debug I need to squeeze after work 😄

@setchy
Copy link
Member

setchy commented Jun 12, 2024

@setchy and @bmulholland I can definitely help... give me a bit, feels more a debug I need to squeeze after work 😄

absolutely! thank you again @LunaticMuch 🙇

@LunaticMuch
Copy link
Author

First debug session, following your guidance @setchy

  1. Console is empty. There's no reported error or warning. Actually nothing at all.
  2. I do not see any call for the image tracing the network... this quite interesting, but it feels weird the security policy. If the connection is not started, this should not be related to a policy. The policy might block the response, not opening a socket for the call

@setchy
Copy link
Member

setchy commented Jun 12, 2024

Looks like you have Detailed Notifications enabled. In that case, you'll find the logic within src/utils/subject.ts that enriches notifications with the most recent SubjectUser (logic varies by notification type ie: Issues vs PullRequests vs Discussions, etc)

@LunaticMuch
Copy link
Author

I need to dig into that, but from my initial testing, it's all fine.
Actually, the repo icon suffers of the same problem, but the default icon for the runners does not 🤔

Screenshot 2024-06-13 at 12 44 03

@setchy
Copy link
Member

setchy commented Jun 13, 2024

The default icons used for workflows (and as a default when no avatar available) are from the primer/octicons package

@setchy
Copy link
Member

setchy commented Jun 19, 2024

Perhaps a fix lies in setting the crossorigin attribute on the img tags...

https://www.geeksforgeeks.org/how-to-allow-cross-origin-use-of-images-and-canvas/

@LunaticMuch - when you said these urls load in a new tab, was that in an incognito/private or a regular instance?

@setchy setchy added the host:github-server Related to GitHub Enterprise Server label Jun 22, 2024
@setchy
Copy link
Member

setchy commented Jul 26, 2024

@LunaticMuch - just wanted to confirm if you're still experiencing this issue?

@setchy setchy added the priority:low Items of low importance. Applicable to select few users or use-cases label Aug 19, 2024
@setchy setchy closed this as not planned Won't fix, can't repro, duplicate, stale Sep 21, 2024
@setchy
Copy link
Member

setchy commented Sep 21, 2024

Closing as stale. Happy to reopen if others report more details to help triage

@mislavperi
Copy link

I can confirm the issue still occurs as of today

@setchy
Copy link
Member

setchy commented Oct 24, 2024

@mislavperi - are you able to share further details please, such as Gitify version, GitHub account type(s), OS type. Any log screenshots would also be useful (via Developer Tools network console or app log file)

@setchy setchy reopened this Oct 25, 2024
@mislavperi
Copy link

@setchy Yes I'll get you those today

@setchy
Copy link
Member

setchy commented Oct 25, 2024

Perfect!

I've also been wondering if these broken avatars also occur on this feature branch which uses the official GitHub Primer avatar components

#1589

@mislavperi
Copy link

mislavperi commented Oct 29, 2024

@setchy
Account type enterprise account, we run on GHE 3.13.3
Running MacOS Sonoma 14.3
Gitlify version is v5.16.1
It also seems to be stuck on high number of notifications (I can't clear them at all) and new ones never come
Screenshot 2024-10-29 at 09 13 58
Screenshot 2024-10-29 at 09 13 11

Here are the inspect element network tab and console errors I get

@setchy
Copy link
Member

setchy commented Oct 31, 2024

HTTP 302 hey... So is there a suggested new location?

If you get the chance to test Gitify locally using that branch I shared, I'd love to hear if this is natively resolved within the GitHub component library

@mislavperi
Copy link

image
@setchy
Still the same. I'm also struggling to open the inspect element for some reason

@setchy
Copy link
Member

setchy commented Nov 4, 2024

Thanks for testing the branch @mislavperi. 🙏

Back to the drawing board then 😅.

I'm curious if this happens for ALL GHE users or a smaller cohort 🤔

@mislavperi
Copy link

Let me know if you need any help @setchy

@setchy
Copy link
Member

setchy commented Nov 4, 2024

We'd love a hand @mislavperi since none of the maintainers are able to recreate this bug and find an effective solution. If you are able to spend some time digging deeper that would be greatly appreciated.


As an aside, I've been thinking about interim "hacks" to address the broken avatar images experience which is not ideal... perhaps we should use the default GitHub Mark Icon avatar (either for ALL GHE accounts, or, through a user setting)... at least until we find the root of the issue

@setchy
Copy link
Member

setchy commented Nov 5, 2024

@pddg - as a GHE user, do you also have this broken avatar issue?

@pddg
Copy link
Contributor

pddg commented Nov 5, 2024

The avatar image was corrupted when a problem occurred connecting to the GHE Server.
Even if the connection is restored, it image will not be restored immediately. However, it will be displayed normally at some point.

Open/Close window, reload manually does not help. Open settings and return to the notifications list will recover the avatar image.
I hadn't really thought about it before, so there may be other cases where it is restored.

@setchy
Copy link
Member

setchy commented Nov 5, 2024

Ah ha, that's helpful insight @pddg.

@mislavperi
Copy link

@setchy In my case, it refuses to connect after initial configuration was defined, I'll spin up the instance locally and see why it might be happening

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working host:github-server Related to GitHub Enterprise Server priority:low Items of low importance. Applicable to select few users or use-cases
Projects
None yet
Development

No branches or pull requests

5 participants