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

Properly escape release attachment URL (#6512) #6523

Merged
merged 1 commit into from Apr 5, 2019

Conversation

4 participants
@mrsdizzie
Copy link
Contributor

commented Apr 5, 2019

Make sure file attachments on a release get a properly escaped URL when
linking.

Fixes #6506

This is backport to release/v1.8

Properly escape release attachment URL (#6512)
Make sure file attachments on a release get a properly escaped URL when
linking.

Fixes #6506

@GiteaBot GiteaBot added the lgtm/need 1 label Apr 5, 2019

@techknowlogick techknowlogick added this to the 1.8.0 milestone Apr 5, 2019

@GiteaBot GiteaBot added lgtm/done and removed lgtm/need 1 labels Apr 5, 2019

@techknowlogick techknowlogick merged commit 2551660 into go-gitea:release/v1.8 Apr 5, 2019

2 checks passed

approvals/lgtm this commit looks good
continuous-integration/drone/pr Build is passing
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.