Skip to content

[Fix #24682]: ensure federation cloud id is retruned if FN property not found - #24683

Merged
juliusknorr merged 1 commit into
nextcloud:masterfrom
doc75:federation_confidentiality
Dec 15, 2020
Merged

[Fix #24682]: ensure federation cloud id is retruned if FN property not found#24683
juliusknorr merged 1 commit into
nextcloud:masterfrom
doc75:federation_confidentiality

Conversation

@doc75

Copy link
Copy Markdown
Contributor

This PR is fixing #24682

I have tested it on my migrated instance (19.0.6) and it is working fine.

Let me know if anything is missing or need to be reworked (I am not an expert)

@doc75
doc75force-pushed the federation_confidentiality branch from 72d77fb to 212df7cCompareDecember 13, 2020 12:19
…roperty not found
Signed-off-by: Guillaume Virlet <github@virlet.org>
@doc75
doc75force-pushed the federation_confidentiality branch from 212df7c to 83491c6CompareDecember 13, 2020 13:00
@kesselbkesselb added 3. to review Waiting for reviews bug labels Dec 14, 2020

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice catch!

@rullzer

Copy link
Copy Markdown
Member

/backport to stable20

@rullzer

Copy link
Copy Markdown
Member

/backport to stable19

@rullzerrullzer added this to the Nextcloud 21 milestone Dec 15, 2020
@rullzerrullzer mentioned this pull request Dec 15, 2020
59 tasks
@juliusknorr
juliusknorr merged commit d37034f into nextcloud:masterDec 15, 2020
@welcome

welcomeBot commented Dec 15, 2020

Copy link
Copy Markdown

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@blizzz

Copy link
Copy Markdown
Member

Nice catch!

yeah. i thought this is a required field.

@rullzer

Copy link
Copy Markdown
Member

/backport to stable20

@rullzer

Copy link
Copy Markdown
Member

/backport to stable19

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable20 failed. Please do this backport manually.

@rullzer

Copy link
Copy Markdown
Member

/backport to stable20

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable20 failed. Please do this backport manually.

blizzz referenced this pull request Jan 5, 2021
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@doc75@rullzer@blizzz@juliusknorr@kesselb