Closed VernonGrant closed 7 months ago
Code Climate has analyzed commit 05589f26 and detected 1 issue on this pull request.
Here's the issue category breakdown:
Category | Count |
---|---|
Complexity | 1 |
View more on Code Climate.
Hi, thanks for the PR. I will try to review by the end of this week.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Hi,
I correct the potential corss site scripting vulnerability, explained at #501 by extracting only the required query parameters during link generation (
lcp_page_link
). All non related query parameters will be automatically discarded. And the remaining page number parameters are maped to their integer values, on failure they are set to1
.I don't have Vagrant on my system, so I wasn't able to run the tests. I did however setup a local site and included a variaty of shortcodes on a single page to insure that page numbers and bookmarks are functioning as needed. Please run the tests and if there's any issue let me know and I'll update the pull request as needed. 🙏
Resolves #501