Skip to content

Verify 'Full Name' field does not allow HTML in Signup form - #16807

Merged
uzairr merged 1 commit into
masterfrom
LEARNER-3385/content-injection-on-registration
Dec 8, 2017
Merged

Verify 'Full Name' field does not allow HTML in Signup form#16807
uzairr merged 1 commit into
masterfrom
LEARNER-3385/content-injection-on-registration

Conversation

@uzairr

@uzairr uzairr commented Dec 6, 2017

Copy link
Copy Markdown
Contributor

Description

'Full Name' field in the signup form is allowing HTML as an input
which makes spoofing easily.To avoid it, validation is added
that will ensure 'Full Name' field does not allow HTML.

LEARNER-3385

@uzairr

uzairr commented Dec 6, 2017

Copy link
Copy Markdown
Contributor Author

jenkins run bokchoy

@uzairr

uzairr commented Dec 7, 2017

Copy link
Copy Markdown
Contributor Author

jenkins run a11y

@uzairr
uzairr force-pushed the LEARNER-3385/content-injection-on-registration branch from 482a1da to 57592b7 Compare December 7, 2017 05:12
@uzairr
uzairr force-pushed the LEARNER-3385/content-injection-on-registration branch 4 times, most recently from 58148f9 to 5070202 Compare December 8, 2017 12:48
'Full Name' field in the signup form is allowing HTML as an input
which makes spoofing easily.To avoid it, validation is added
that will ensure 'Full Name' field does not allow HTML.

LEARNER-3385
@uzairr
uzairr force-pushed the LEARNER-3385/content-injection-on-registration branch from 5070202 to 8ffac20 Compare December 8, 2017 14:18

@waheedahmed waheedahmed left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure all tests passes.

@uzairr
uzairr merged commit 1225bde into master Dec 8, 2017
@uzairr
uzairr deleted the LEARNER-3385/content-injection-on-registration branch December 8, 2017 16:34
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production on Monday, December 11, 2017.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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 this pull request may close these issues.

4 participants