fix: don't tell the remote their token is lower#60547

Merged

kesselb merged 2 commits into

fed-share-secret-lower-dont-tell

May 20, 2026

Member

icewind1991 commented May 19, 2026

Resolves: #

Summary

TODO

...

Checklist

Code is properly formatted
Sign-off message is added to all commits
Tests (unit, integration, api and/or acceptance) are included
Screenshots before/after for front-end changes
Documentation (manuals or wiki) has been updated or is not required
Backports requested where applicable (ex: critical bugfixes)
Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
Milestone added for target branch/version (ex: 32.x for stable32)

AI (if applicable)

The content of this PR was partly or fully generated using AI


          fix: don't tell the remote their token is lower

705abc2

Signed-off-by: Robin Appelman <robin@icewind.nl>

icewind1991 added this to the Nextcloud 35 milestone

icewind1991 requested a review from a team as a code owner

May 19, 2026 15:09

icewind1991 requested review from Altahrim, ArtificialOwl, come-nc and salmart-dev and removed request for a team

May 19, 2026 15:09

icewind1991 added the 3. to review label

Member Author

icewind1991 commented May 19, 2026

/backport! to stable34

backportbot Bot added the backport-request label

backportbot Bot mentioned this pull request

[stable34] fix: don't tell the remote their token is lower #60573

Merged

backportbot Bot removed the backport-request label

Member Author

icewind1991 commented May 19, 2026

/backport! to stable33

Member Author

icewind1991 commented May 19, 2026

/backport! to stable32

backportbot Bot added the backport-request label

backportbot Bot mentioned this pull request

[stable33] fix: don't tell the remote their token is lower #60574

Merged

backportbot Bot removed the backport-request label

backportbot Bot mentioned this pull request

[stable32] fix: don't tell the remote their token is lower #60575

Merged

Member Author

icewind1991 commented May 19, 2026

/backport! to stable31

Member Author

icewind1991 commented May 19, 2026

/backport! to stable30

backportbot Bot added the backport-request label

Member Author

icewind1991 commented May 19, 2026

/backport! to stable29

backportbot Bot mentioned this pull request

[stable31] fix: don't tell the remote their token is lower #60576

Merged

backportbot Bot removed the backport-request label

This was referenced May 19, 2026

[stable30] fix: don't tell the remote their token is lower #60577

Merged

[stable29] fix: don't tell the remote their token is lower #60578

Merged

Member Author

icewind1991 commented May 19, 2026

/backport! to stable28

Member Author

icewind1991 commented May 19, 2026

/backport! to stable27

backportbot Bot added the backport-request label

Member Author

icewind1991 commented May 19, 2026

/backport! to stable26

backportbot Bot mentioned this pull request

[stable28] fix: don't tell the remote their token is lower #60579

Merged

backportbot Bot removed the backport-request label

This was referenced May 19, 2026

[stable27] fix: don't tell the remote their token is lower #60580

Merged

[stable26] fix: don't tell the remote their token is lower #60581

Merged

Member Author

icewind1991 commented May 19, 2026

/backport! to stable25

backportbot Bot added the backport-request label

backportbot Bot mentioned this pull request

[stable25] fix: don't tell the remote their token is lower #60582

Open

Member Author

icewind1991 commented May 19, 2026

/backport! to stable24

Member Author

icewind1991 commented May 19, 2026

/backport! to stable23

backportbot Bot removed the backport-request label

Member Author

icewind1991 commented May 19, 2026

/backport! to stable22

Member Author

icewind1991 commented May 19, 2026

/backport! to stable21

backportbot Bot added the backport-request label

backportbot Bot mentioned this pull request

[stable24] fix: don't tell the remote their token is lower #60583

Open

backportbot Bot removed the backport-request label

This was referenced May 19, 2026

[stable23] fix: don't tell the remote their token is lower #60584

Open

[stable22] fix: don't tell the remote their token is lower #60585

Open

[stable21] fix: don't tell the remote their token is lower #60586

Open


          test: adjust tests

e9fc60b

Signed-off-by: Robin Appelman <robin@icewind.nl>

susnux added the bug label

kesselb approved these changes

View reviewed changes

kesselb enabled auto-merge

May 19, 2026 21:45

CarlSchwan approved these changes

View reviewed changes

kesselb merged commit 9a00604 into master

193 of 200 checks passed

kesselb deleted the fed-share-secret-lower-dont-tell branch

May 20, 2026 07:48

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

Reviewers

kesselb kesselb approved these changes

CarlSchwan CarlSchwan approved these changes

ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl is a code owner automatically assigned from nextcloud/server-backend

Altahrim Awaiting requested review from Altahrim Altahrim is a code owner automatically assigned from nextcloud/server-backend

salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend

come-nc Awaiting requested review from come-nc come-nc is a code owner automatically assigned from nextcloud/server-backend

Labels

3. to review bug