Skip to content

Conversation

@nfebe
Copy link
Contributor

@nfebe nfebe commented Oct 6, 2025

The password param should never be sent if the intention is not remove it or update it.

This commit adapts the frontend and backend to this rule to avoid weird bugs especially around updating new shares.

@nfebe nfebe requested review from a team as code owners October 6, 2025 15:14
@nfebe nfebe added the 3. to review Waiting for reviews label Oct 6, 2025
@nfebe nfebe requested review from icewind1991 and removed request for a team October 6, 2025 15:14
@nfebe
Copy link
Contributor Author

nfebe commented Oct 6, 2025

/backport to stable32

@nfebe
Copy link
Contributor Author

nfebe commented Oct 6, 2025

/backport to stable31

Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

Code makes sense but did not test it

Copy link
Contributor

@artonge artonge left a comment

Choose a reason for hiding this comment

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

Looks good beside nitpick.

@nfebe nfebe force-pushed the fix/share-permissions-password-validation branch from a9b74c0 to 425d455 Compare October 20, 2025 13:55
@nfebe
Copy link
Contributor Author

nfebe commented Oct 20, 2025

/compile

@nfebe nfebe enabled auto-merge October 20, 2025 13:55
@nfebe nfebe force-pushed the fix/share-permissions-password-validation branch from b1799aa to 5d726aa Compare October 20, 2025 15:52
@nfebe
Copy link
Contributor Author

nfebe commented Oct 20, 2025

/compile

@nfebe nfebe force-pushed the fix/share-permissions-password-validation branch from 5d726aa to 7b31bf1 Compare October 20, 2025 15:59
@nfebe
Copy link
Contributor Author

nfebe commented Oct 20, 2025

/compile

The password param should never be sent if the intention is not
remove it or update it.

This commit adapts the frontend and backend to this rule to avoid weird bugs
especially around updating new shares.

Signed-off-by: nfebe <[email protected]>
@nfebe nfebe force-pushed the fix/share-permissions-password-validation branch from 06eaddc to a28834b Compare October 21, 2025 15:55
@nfebe
Copy link
Contributor Author

nfebe commented Oct 21, 2025

/compile

Signed-off-by: nextcloud-command <[email protected]>
@nfebe nfebe merged commit 87b94c7 into master Oct 21, 2025
205 of 214 checks passed
@nfebe nfebe deleted the fix/share-permissions-password-validation branch October 21, 2025 20:08
@nfebe
Copy link
Contributor Author

nfebe commented Oct 24, 2025

/backport to stable32

@nfebe
Copy link
Contributor Author

nfebe commented Oct 24, 2025

/backport to stable31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants