Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decode URL for chinese file name #417

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FullStackEngN
Copy link

If URL contains Chinese file name, then URL will be encode.
So when the server get the URL, it need decode it firstly.

For example,

If user send this URL:
http://sp16:8081/wopi/files/工作簿1.xlsx?access_token=123

Then browser will encode it, and server get this URL:
http://sp16:8081/wopi/files/%E5%B7%A5%E4%BD%9C%E7%B0%BF1.xlsx?access_token=123

So server need decode it to get the right file name.

@FullStackEngN
Copy link
Author

Read the doc again, and found the file id should only contain letters and numbers, so Chinese file name is not legal here.
Maybe this is not a good pull request, but if we add this, then it's easy for test purpose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant