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

Set/Validate/escape these parameters in the settings page #172

Merged
merged 1 commit into from
Jan 31, 2017

Conversation

donnchawp
Copy link
Contributor

Avoid XSS by validating and escape these parameters on the settings
page. The XSS is on an admin-only page so impact is low. Only an
admin user can set it.
Also set the ossdl_https index to avoid a php warning on the CDN tab.

Avoid XSS by validating and escape these parameters on the settings
page. The XSS is on an admin-only page so impact is low.
Also set the ossdl_https index to avoid a php warning on the CDN tab.
@donnchawp donnchawp added this to the 1.4.9 milestone Jan 30, 2017
@donnchawp donnchawp self-assigned this Jan 30, 2017
@donnchawp donnchawp requested a review from kraftbj January 30, 2017 18:11
@donnchawp donnchawp merged commit 99037db into master Jan 31, 2017
@donnchawp donnchawp deleted the fix_xss_on_settings_page branch January 31, 2017 16:37
kraftbj pushed a commit that referenced this pull request Feb 9, 2017
* Don't delete so many files on blogs with static homepages. #175
* Discourage use of file locking because semaphores cause problems and it's not needed. #174
* Reorganise the settings page in small ways #173
* Fixed XSS in settings page. Props @jaypatel #172



git-svn-id: http://plugins.svn.wordpress.org/wp-super-cache/trunk@1587563 b8457f37-d9ea-0310-8a92-e5e31aec5664
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.

1 participant