alluxio icon indicating copy to clipboard operation
alluxio copied to clipboard

Published 20 hours ago • verified publisher icon Alluxio

Parse the URI if needed when creating the signature

Open Haoning-Sun opened this issue 1 year ago • 4 comments

What changes are proposed in this pull request?

Fix #18442.

Why are the changes needed?

Using URI. getPath() will decode the original path.

Haoning-Sun avatar Nov 24 '23 13:11 Haoning-Sun

The test result is as follows, and the decoded URI is required to calculate the signature. image

Haoning-Sun avatar Nov 24 '23 13:11 Haoning-Sun

Automated checks report:

  • PR title follows the conventions: FAIL
    • The title of the PR does not pass all the checks. Please fix the following issues:
      • First word of title ("Decode") is not an imperative verb. Please use one of the valid words
  • Commits associated with Github account: PASS

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

alluxio-bot avatar Nov 24 '23 13:11 alluxio-bot

Automated checks report:

  • PR title follows the conventions: PASS
  • Commits associated with Github account: PASS

All checks passed!

alluxio-bot avatar Dec 05 '23 01:12 alluxio-bot

@Jackson-Wang-7 This pr fixes the issue with dora, just like #18444.

Haoning-Sun avatar Dec 13 '23 00:12 Haoning-Sun