BioDrop icon indicating copy to clipboard operation
BioDrop copied to clipboard

Found two user profiles whose GitHub profiles does not exist

Open Aadarsh805 opened this issue 3 years ago • 11 comments

Description

  1. Skyhero-admin.json
  2. aaquibdev.json

this error occurred due to this upstream image response failed for https://github.com/Skyhero-admin.png 404

Screenshots

this caused the random user section to generate only 4 user profiles, the 5th threw error Screenshot from 2023-01-09 00-40-14

Additional information

There must be more accounts like this, it might be negligible for a large number of users (this happened on my local dev server, with smaller user list)

Aadarsh805 avatar Jan 08 '23 19:01 Aadarsh805

It's great having you contribute to this project

Welcome to the community :nerd_face:

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

github-actions[bot] avatar Jan 08 '23 19:01 github-actions[bot]

Thank you raising 👍

eddiejaoude avatar Jan 09 '23 14:01 eddiejaoude

Description

  1. Skyhero-admin.json
  2. aaquibdev.json

this error occurred due to this upstream image response failed for https://github.com/Skyhero-admin.png 404

Screenshots

this caused the random user section to generate only 4 user profiles, the 5th threw error Screenshot from 2023-01-09 00-40-14

Additional information

There must be more accounts like this, it might be negligible for a large number of users (this happened on my local dev server, with smaller user list)

@Aadarsh805 I also tried with dummy database with some users having GitHub profile and some not, but in that case it was using the fallbackSvg avatar instead of breaking from that point like as you mentioned because of unavailability 4th user 5th one was not displayed, I dont know if I'am getting it right or not 😅.

MD-ARBAB-SAHID avatar Jan 11 '23 16:01 MD-ARBAB-SAHID

Description

  1. Skyhero-admin.json
  2. aaquibdev.json

this error occurred due to this upstream image response failed for https://github.com/Skyhero-admin.png 404

Screenshots

this caused the random user section to generate only 4 user profiles, the 5th threw error Screenshot from 2023-01-09 00-40-14

Additional information

There must be more accounts like this, it might be negligible for a large number of users (this happened on my local dev server, with smaller user list)

@Aadarsh805 I also tried with dummy database with some users having GitHub profile and some not, but in that case it was using the fallbackSvg avatar instead of breaking from that point like as you mentioned because of unavailability 4th user 5th one was not displayed, I dont know if I'am getting it right or not sweat_smile.

The one I'm talking about here don't have a GitHub account, not just the profile picture If that's what you mean my having fallbackSvg in place of GitHub profile picture

Aadarsh805 avatar Jan 11 '23 16:01 Aadarsh805

@Aadarsh805 Even if the account is not there in github but if there is a json file in data dir in that case fallbackSvg will be used. It only displays on the console unable to find .png 404 but it does not decrease the number of profiles. Tried with some dummy profiles without Github account.

MD-ARBAB-SAHID avatar Jan 12 '23 19:01 MD-ARBAB-SAHID

@Aadarsh805 Even if the account is not there in github but if there is a json file in data dir in that case fallbackSvg will be used. It only displays on the console unable to find .png 404 but it does not decrease the number of profiles. Tried with some dummy profiles without Github account.

Huh then that's weird, this is the error on my terminal This one's causing that, not the 404 one image

Aadarsh805 avatar Jan 12 '23 19:01 Aadarsh805

@Aadarsh805 Even if the account is not there in github but if there is a json file in data dir in that case fallbackSvg will be used. It only displays on the console unable to find .png 404 but it does not decrease the number of profiles. Tried with some dummy profiles without Github account.

Huh then that's weird, this is the error on my terminal This one's causing that, not the 404 one image

Then ig this error mainly occurs if profile is stored in mongodb database and the corresponding profile.json is not present in data directory irrespective of whether GitHub account exist or not. I created dummy user1,user2,user3,user4 which don't have a GitHub account but their data is stored in mongodb database but corresponding json files are not present in data dir. Screenshot 2023-01-15 at 12 01 14 AM For user with username "HAT52" who has a Github account along with the "HAT52.json" stored in data dir. I deleted "HAT52.json" stored in data dir in that case it shows: Screenshot 2023-01-15 at 12 08 33 AM

@eddiejaoude So ig error mainly occurs when data is present in the database and corresponding profile_data.json file is not present in data dir which leads to data inconsistency.

MD-ARBAB-SAHID avatar Jan 14 '23 18:01 MD-ARBAB-SAHID

Great collaboration 👍

Yes if the user file is deleted ,they remain in the database - this is something we need to fix

I suggest if the file is not found, the user is removed from the database

eddiejaoude avatar Feb 03 '23 15:02 eddiejaoude

These profiles do exist? 🤔

  • Skyhero-admin.json https://github.com/EddieHubCommunity/LinkFree/blob/main/data/Skyhero-admin.json
  • aaquibdev.json https://github.com/EddieHubCommunity/LinkFree/blob/main/data/aaquibdev.json

eddiejaoude avatar Feb 03 '23 15:02 eddiejaoude

We can use a cron for that. Delete users from DB whose json file is not available. I would like to work on this cron job issue. If yes, let me know. i can create a issue in relation to this issue.

psavan1655 avatar Mar 03 '23 06:03 psavan1655

we can do one more like if user file is deleted then changes the UI { this user is not available }

Screenshot from 2023-03-04 12-50-56

something like that we can improve UI It's just an example

Tusharjagi avatar Mar 04 '23 07:03 Tusharjagi

I will look into this.

SaraJaoude avatar Apr 13 '23 05:04 SaraJaoude

@Skyhero-alt I believe that the account Skyhero-admin is your previous account. Can you please comment on this Issue so that we can address it and close it? You can either edit your LinkFree profile or close this if you decide you no longer require it.

SaraJaoude avatar Apr 27 '23 04:04 SaraJaoude

With aaquibdev.json the GitHub username does not exist and none of the links work

SaraJaoude avatar Apr 27 '23 04:04 SaraJaoude

Yes it is my old account I think i will update the link Should I make a Pull request for it ?

Skyhero-alt avatar May 04 '23 18:05 Skyhero-alt

@Skyhero-alt yes please 👍

eddiejaoude avatar May 05 '23 02:05 eddiejaoude

With aaquibdev.json the GitHub username does not exist and none of the links work

From the commit history, I found that it's the previous account of @imaaquibali.

@imaaquibali, Since you have created a new LinkFree profile on your new username, could you please delete your old json file: https://github.com/EddieHubCommunity/LinkFree/blob/main/data/aaquibdev.json or we should address it?

kumarsonsoff3 avatar May 07 '23 08:05 kumarsonsoff3

Yes it is my old account I think i will update the link Should I make a Pull request for it ?

This has now been dealt with in: https://github.com/EddieHubCommunity/LinkFree/pull/6716

SaraJaoude avatar May 10 '23 04:05 SaraJaoude

With aaquibdev.json the GitHub username does not exist and none of the links work

From the commit history, I found that it's the previous account of @imaaquibali.

@imaaquibali, Since you have created a new LinkFree profile on your new username, could you please delete your old json file: https://github.com/EddieHubCommunity/LinkFree/blob/main/data/aaquibdev.json or we should address it?

I will delete the file aaquibdev.json as:

  • it has been two weeks and there has been no response from @imaaquibali
  • the GitHub username does not exist
  • the links mentioned in the file do not work

SaraJaoude avatar May 19 '23 04:05 SaraJaoude

@SaraJaoude Actualy I don't have access to my @imaaquibali account 😞. I lost my backup codes and forget to take backup of 2FA also.

If you know any other options to access my account please let me know.

I thought I already deleted my old profile btw thanks for deleting it 😊

aliaaquib avatar May 19 '23 07:05 aliaaquib