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: Create file deletion request upon revoke storage request without priority challenge #353

Merged
merged 10 commits into from
Feb 13, 2025

Conversation

snowmead
Copy link
Contributor

@snowmead snowmead commented Feb 4, 2025

Create a file deletion request when a storage request has been revoked while an MSP has already confirmed storing the file.

Without this, there was no way to force MSPs from deleting the file from their forest and this allowed them to continue charging for the file.

Added an additional flag to the do_delete_file to specify whether or not to queue a priority challenge to avoid challenging the BSPs twice since we already issue a priority challenge in the cleanup_storage_request.

@santikaplan santikaplan requested a review from TDemeco February 6, 2025 13:38
@santikaplan santikaplan requested a review from ffarall February 11, 2025 18:10
Copy link
Contributor

@ffarall ffarall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job!

@ffarall ffarall merged commit fb3f7e6 into main Feb 13, 2025
25 checks passed
@ffarall ffarall deleted the fix/delete-file-without-queuing-priority branch February 13, 2025 22:04
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.

2 participants