Skip to content

Handle external share with invalid host - #30353

Merged
CarlSchwan merged 1 commit into
masterfrom
fix/localhost-external-share
Dec 29, 2021
Merged

Handle external share with invalid host#30353
CarlSchwan merged 1 commit into
masterfrom
fix/localhost-external-share

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

remoteIsOwnCloud might throw an exception when the host is localhost.
Handle this case instead of aborting completely. The behavior is the
same as that is done 10 lines under it

Signed-off-by: Carl Schwan carl@carlschwan.eu

@CarlSchwanCarlSchwan added the 3. to review Waiting for reviews label Dec 20, 2021
@CarlSchwanCarlSchwan added this to the Nextcloud 24 milestone Dec 20, 2021
@CarlSchwan
CarlSchwan requested review from a team and ArtificialOwlDecember 20, 2021 14:23
@CarlSchwanCarlSchwan self-assigned this Dec 20, 2021
@CarlSchwan
CarlSchwan requested review from icewind1991 and skjnldsv and removed request for a teamDecember 20, 2021 14:23
remoteIsOwnCloud might throw an exception when the host is localhost.
Handle this case instead of aborting completely. The behavior is the
same as that is done 10 lines under it
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwanforce-pushed the fix/localhost-external-share branch from 234d6df to 325be66CompareDecember 20, 2021 14:28
@CarlSchwan
CarlSchwan merged commit 910f05a into masterDec 29, 2021
@CarlSchwan
CarlSchwan deleted the fix/localhost-external-share branch December 29, 2021 16:44
@CarlSchwan

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@CarlSchwan

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@CarlSchwan

Copy link
Copy Markdown
MemberAuthor

/backport to stable21

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

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@CarlSchwan@artonge@skjnldsv