Skip to content

Ask server to recalculate checksum(hash)#4100

Merged
allexzander merged 5 commits intomasterfrom
feature/ask-server-to-recalc-checksum
Jan 11, 2022
Merged

Ask server to recalculate checksum(hash)#4100
allexzander merged 5 commits intomasterfrom
feature/ask-server-to-recalc-checksum

Conversation

@allexzander
Copy link
Contributor

@allexzander allexzander commented Dec 20, 2021 •
edited
Loading

As nextcloud/server#27378 is already merged and is going to be released in NC24.

This will fix #3409

@allexzander allexzander changed the title Feature/ask server to recalc checksum Ask server to recalculate checksum(hash) Dec 20, 2021
@szaimen
Copy link
Contributor

szaimen commented Dec 20, 2021 •
edited
Loading

As nextcloud/server#27378 is already merged and is going to be released in NC24.

No it is not yet. See nextcloud/server#29510

mgallien
mgallien reviewed Dec 22, 2021
View reviewed changes
Copy link
Collaborator

@mgallien mgallien left a comment

Choose a reason for hiding this comment

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

nice to have automated tests !
see my comments

@allexzander allexzander force-pushed the feature/ask-server-to-recalc-checksum branch 3 times, most recently from addd911 to fa6227d Compare December 23, 2021 10:39
@allexzander allexzander requested a review from mgallien December 23, 2021 10:39
@allexzander
Copy link
Contributor Author

allexzander commented Dec 23, 2021

As nextcloud/server#27378 is already merged and is going to be released in NC24.

No it is not yet. See nextcloud/server#29510

@szaimen Can I still assume this feature to be available in 24?

@szaimen
Copy link
Contributor

szaimen commented Dec 23, 2021

@szaimen Can I still assume this feature to be available in 24?

I don't know. Feel free to ask this in the files or engineering chat :)

@allexzander
Copy link
Contributor Author

allexzander commented Dec 23, 2021

nice to have automated tests ! see my comments

@mgallien Thank you. Fixed.

@allexzander allexzander force-pushed the feature/ask-server-to-recalc-checksum branch from 27848cb to 5b9a5ce Compare December 27, 2021 08:27
@allexzander
Copy link
Contributor Author

allexzander commented Jan 10, 2022

@mgallien Any comments?

mgallien
mgallien reviewed Jan 11, 2022
View reviewed changes
@allexzander allexzander force-pushed the feature/ask-server-to-recalc-checksum branch from 6632c74 to 1b33852 Compare January 11, 2022 12:42
@allexzander
Copy link
Contributor Author

allexzander commented Jan 11, 2022

/rebase

@nextcloud-command nextcloud-command force-pushed the feature/ask-server-to-recalc-checksum branch from 1b33852 to 4ced494 Compare January 11, 2022 12:45
@allexzander
Copy link
Contributor Author

allexzander commented Jan 11, 2022

/rebase

@allexzander
Ask server to recalculate checksum on validatin failure.
b7be10f 
Signed-off-by: alex-z <blackslayer4@gmail.com>
@allexzander
Fix typo in usernamePrefillServerVersionMinSupportedMajor.
6f5f3e7 
Signed-off-by: alex-z <blackslayer4@gmail.com>
@allexzander
Checksum validator unit tests.
5b0e2d8 
Signed-off-by: alex-z <blackslayer4@gmail.com>
@allexzander
Checksum validation PropagateDownload unit tests.
190d278 
Signed-off-by: alex-z <blackslayer4@gmail.com>
@allexzander
Refactor DeleteJob and DeleteApiJob to use SimpleFileJob.
225753a 
Signed-off-by: alex-z <blackslayer4@gmail.com>
@nextcloud-command nextcloud-command force-pushed the feature/ask-server-to-recalc-checksum branch from 4ced494 to 225753a Compare January 11, 2022 13:37
@allexzander allexzander merged commit 2e0ba82 into master Jan 11, 2022
@allexzander allexzander deleted the feature/ask-server-to-recalc-checksum branch January 11, 2022 13:40
@allexzander
Copy link
Contributor Author

allexzander commented Jan 11, 2022

this can wait till next major release

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 11, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 18 Code Smells

83.3% 83.3% Coverage
0.0% 0.0% Duplication

@nextcloud-desktop-bot
Copy link

nextcloud-desktop-bot commented Jan 11, 2022

AppImage file: Nextcloud-PR-4100-225753a8c0c45a3c2866f5b6a0cfa7af1933f498-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@mgallien mgallien added this to the 3.5.0 milestone Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mgallien mgallien mgallien approved these changes

@claucambra claucambra claucambra approved these changes

@camilasan camilasan Awaiting requested review from camilasan camilasan is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.5.0

Development

Successfully merging this pull request may close these issues.

Add checksum recalculate request from the client side after corresponding server-side PR is merged and released

5 participants

@allexzander @szaimen @nextcloud-desktop-bot @mgallien @claucambra