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

Fix to be able to redownload avatar and header #16190

Merged

Conversation

noellabo
Copy link
Contributor

@noellabo noellabo commented May 9, 2021

#15477 Take 2.

If the acquisition of avatar and header fails, the already acquired image will be discarded. This solves the problem that you can't update your avatar because you're holding the wrong image.

Perform RedownloadAvatarWorker and RedownloadHeaderWorker if you cannot get the avatar or header from the specified url due to a media server issue when updating your account information.

The Worker will retry 7 times. This is about a day. From the next day onward, when the account information is referenced, the old account information will be updated, so there will be a chance that the image will be updated again.

@noellabo
Copy link
Contributor Author

noellabo commented May 9, 2021

The code hasn't been tested yet, but how about this approach?

@noellabo noellabo force-pushed the fix-to-be-able-to-redownload-avatar-and-header branch from e3a0b1e to ec30886 Compare May 9, 2021 11:19
@noellabo noellabo force-pushed the fix-to-be-able-to-redownload-avatar-and-header branch from ec30886 to 43a0cfa Compare May 11, 2021 02:22
@noellabo noellabo marked this pull request as ready for review May 11, 2021 02:39
@noellabo
Copy link
Contributor Author

Looks like it's working.

@Gargron Gargron merged commit c403c36 into mastodon:main May 11, 2021
chrisguida pushed a commit to Start9Labs/mastodon that referenced this pull request Feb 26, 2022
* Fix to reset if header and avatar download fails

* Add RedownloadAvatarWorker and RedownloadHeaderWorker
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants