Skip to content
This repository was archived by the owner on Dec 30, 2025. It is now read-only.

XSS, CSRF replace#246

Open
lispro06 wants to merge 1 commit into4clojure:developfrom
lispro06:patch-1
Open

XSS, CSRF replace#246
lispro06 wants to merge 1 commit into4clojure:developfrom
lispro06:patch-1

Conversation

@lispro06
Copy link

@lispro06 lispro06 commented Sep 9, 2013

security vulnerability remove

security vulnerability remove
@lispro06
Copy link
Author

lispro06 commented Sep 9, 2013

below the images show that approving problem using approved user's authority.

'view unapproved problems' menu is enabled for approved user(s).

When an approved user click CSRF problem, approve value(#) problem(s).

unapproved
approved
fixed

tested in other site for study.

@lispro06
Copy link
Author

lispro06 commented Sep 9, 2013

<form name="csrf" action="http://~/problem/reject" method="post" target="hidden">
<input type="hidden" name="id" value="91" />
</form>
<script>document.csrf.submit();</script>

above CSRF is remove value(91) problem using approved user's authority.

@amcnamara
Copy link
Member

While I'm inclined to agree that rendering HTML into the problem title/description etc is super bad form, many of the problems already have markup in their descriptions. We would need to edit them all before this change could land.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants