Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

gcs: fix url parse panic (#664) #673

Merged
merged 1 commit into from
Jan 8, 2021

Conversation

ti-srebot
Copy link
Contributor

cherry-pick #664 to release-4.0


What problem does this PR solve?

Resolve: #661

What is changed and how it works?

Allow gcs prefix is empty. like s3.

Check List

Tests

  • Manual test (add detailed scripts or steps below)

Related changes

  • Need to cherry-pick to the release branch

Release Note

  • Fix the issue that br paniced when gcs storage url has not prefix.

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
Copy link
Collaborator

@kennytm kennytm left a comment

Choose a reason for hiding this comment

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

lgtm

@glorv
Copy link
Collaborator

glorv commented Jan 8, 2021

LGTM

@ti-srebot ti-srebot added status/LGT2 LGTM2 and removed status/LGT1 LGTM1 labels Jan 8, 2021
@kennytm kennytm merged commit 707abab into pingcap:release-4.0 Jan 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants