XSS fix for bunny1 #1

Merged
merged 1 commit into from Apr 30, 2012

Conversation

Projects
None yet
2 participants
@alokmenghrajani
Contributor

alokmenghrajani commented Apr 30, 2012

Hey Cheever, how's it going? Are you still in the bay area?

Here is a quick fix for bunny1. It's tested and all...

@ccheever

This comment has been minimized.

Show comment
Hide comment
@ccheever

ccheever Apr 30, 2012

Owner

hey-
thanks for this patch!
i'll do a pull request as soon as i get a chance.

i'm still in the bay area, in palo alto. office is at ramona and
hamilton. are you at fb ?
c

On Mon, Apr 30, 2012 at 4:09 PM, Alok Menghrajani <
reply@reply.github.com

wrote:

Hey Cheever, how's it going? Are you still in the bay area?

Here is a quick fix for bunny1. It's tested and all...

You can merge this Pull Request by running:

git pull https://github.com/alokmenghrajani/bunny1 master

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

  • Fixes an XSS in bunny1.

-- File Changes --

M src/bunny1.py (2)

-- Patch Links --

https://github.com/ccheever/bunny1/pull/1.patch
https://github.com/ccheever/bunny1/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

Owner

ccheever commented Apr 30, 2012

hey-
thanks for this patch!
i'll do a pull request as soon as i get a chance.

i'm still in the bay area, in palo alto. office is at ramona and
hamilton. are you at fb ?
c

On Mon, Apr 30, 2012 at 4:09 PM, Alok Menghrajani <
reply@reply.github.com

wrote:

Hey Cheever, how's it going? Are you still in the bay area?

Here is a quick fix for bunny1. It's tested and all...

You can merge this Pull Request by running:

git pull https://github.com/alokmenghrajani/bunny1 master

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

  • Fixes an XSS in bunny1.

-- File Changes --

M src/bunny1.py (2)

-- Patch Links --

https://github.com/ccheever/bunny1/pull/1.patch
https://github.com/ccheever/bunny1/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

ccheever added a commit that referenced this pull request Apr 30, 2012

@ccheever ccheever merged commit e84376b into ccheever:master Apr 30, 2012

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