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

XSS from GitHub <script>alert(1)</script> #5656

Closed
marsrobertson opened this issue Dec 19, 2019 · 14 comments · Fixed by #5688
Closed

XSS from GitHub <script>alert(1)</script> #5656

marsrobertson opened this issue Dec 19, 2019 · 14 comments · Fixed by #5688

Comments

@marsrobertson
Copy link

https://gitcoin.co/issue/kleros/hackathon/1/2824

image

It fetches content from GitHub issue, that has some XSS...

It only took me half a year to report.

@gitcoinbot
Copy link
Member

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 0.4 ETH (50.67 USD @ $126.68/ETH) attached to it.

@gitcoinbot
Copy link
Member

⚡️ A tip worth 0.20000 ETH (25.34 USD @ $126.68/ETH) has been granted to @marsrobertson for this issue from @owocki. ⚡️

Nice work @marsrobertson! Your tip has automatically been deposited in the ETH address we have on file.

@gitcoinbot
Copy link
Member

Bug Finder ⚡️ A *Bug Finder* Kudos has been sent to @marsrobertson for this issue from @owocki. ⚡️

Nice work @marsrobertson!
Your Kudos has automatically been sent in the ETH address we have on file.

@gitcoinbot
Copy link
Member

gitcoinbot commented Dec 19, 2019

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 1 week, 5 days from now.
Please review their action plans below:

1) mul53 has been approved to start work.

Hi, i have looked around and found one of the areas not been escaped is the Submit A Plan field. I will look around more and make fixes if approved.

Learn more on the Gitcoin Issue Details page.

@marsrobertson
Copy link
Author

That is very kind, didn't expect a tip!

(yeah, it took only HALF YEAR to report as a bug)

@mul53
Copy link
Contributor

mul53 commented Dec 22, 2019

@owocki, i want to create a bounty in the test environment. It says Please enable this token to proceed.. On the /settings/token page under tokens there is no ETH. What steps should i take?

@mul53
Copy link
Contributor

mul53 commented Dec 22, 2019

Hey @owocki is there a reason this middleware was commented out, when its included the input entered by users is escaped which prevents a XSS attack.

# 'app.middleware.bleach_requests',

@kuhnchris
Copy link
Contributor

I think it caused issues with other parts of the page, @thelostone-mc @danlipert you guys remember why we disabled that?

@danlipert
Copy link
Contributor

@kuhnchris @mul53 yeah, it was causing issues in quite a few places so we ended up disabling it

@gitcoinbot
Copy link
Member

@mul53 Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@mul53
Copy link
Contributor

mul53 commented Dec 26, 2019

Alright, i will just add pre_save callback to escape fields before save

@gitcoinbot
Copy link
Member

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 0.4 ETH (51.69 USD @ $129.23/ETH) has been submitted by:

  1. @mul53

@owocki please take a look at the submitted work:


@mul53
Copy link
Contributor

mul53 commented Jan 6, 2020

Hey what’s the status on this, I’m kinda locked up on the number of bounties I can pick up.

@gitcoinbot
Copy link
Member

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The funding of 0.4 ETH (56.22 USD @ $140.54/ETH) attached to this issue has been approved & issued to @mul53.

Additional Tips for this Bounty:

  • owocki tipped 0.2000 ETH worth 28.11 USD to marsrobertson.

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 a pull request may close this issue.

5 participants