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.

Sign up for GitHub

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

Jump to bottom

Use a token when escaping the add_order_item query. props Jon Cave. #33

Merged
merged 1 commit into from
Sep 30, 2011
Merged

Use a token when escaping the add_order_item query. props Jon Cave. #33

merged 1 commit into from
Sep 30, 2011

Conversation

markjaquith
Copy link
Contributor

It's not enough to just use $wpdb->prepare() — you need to actually use the %s tokens!

markjaquith added a commit that referenced this pull request Sep 30, 2011
@markjaquith
Merge pull request #33 from markjaquith/security
2a5d731 
Use a token when escaping the add_order_item query. props @joncave
@markjaquith markjaquith merged commit 2a5d731 into woocommerce:master Sep 30, 2011
@mikejolley
Copy link
Member

I feel dirty for missing that :/

Thanks

@kosmar kosmar mentioned this pull request Jun 10, 2016
Closed
@umeshnevase umeshnevase mentioned this pull request Jun 13, 2016
Closed
@scottbuscemi scottbuscemi mentioned this pull request Jun 20, 2016
Closed
@cruzjoey0507 cruzjoey0507 mentioned this pull request Jun 26, 2016
Closed
@EugenePodovinnikov EugenePodovinnikov mentioned this pull request Aug 18, 2016
Closed
@seb86 seb86 mentioned this pull request Jan 17, 2017
Closed
@iman707 iman707 mentioned this pull request Apr 20, 2017
Closed
This was referenced May 15, 2017
Closed
Closed
@TAGarrison TAGarrison mentioned this pull request Aug 15, 2017
Closed
@prageshs prageshs mentioned this pull request Sep 14, 2017
Closed
@Tommo1973 Tommo1973 mentioned this pull request Nov 20, 2017
Closed
4 tasks
@jdillman1989 jdillman1989 mentioned this pull request Jan 23, 2018
Closed
7 tasks
@johnlmccurdy johnlmccurdy mentioned this pull request Feb 1, 2018
Closed
7 tasks
@AbdulSamadKt AbdulSamadKt mentioned this pull request Feb 12, 2018
Closed
7 tasks
@rwebster85 rwebster85 mentioned this pull request May 26, 2018
Closed
@seb86 seb86 mentioned this pull request Jun 6, 2018
Closed
3 tasks
@maciej-laskowski maciej-laskowski mentioned this pull request Sep 27, 2019
Open
2 tasks
@amber090 amber090 mentioned this pull request Apr 27, 2020
Closed
@LichtWesenNL LichtWesenNL mentioned this pull request Oct 10, 2020
Closed
5 tasks
@panaloptec panaloptec mentioned this pull request Dec 20, 2020
Closed
@studiojpeg studiojpeg mentioned this pull request Mar 29, 2021
Closed
@angelitodiaz1 angelitodiaz1 mentioned this pull request Nov 1, 2021
Closed
5 tasks
@SebastianTusk SebastianTusk mentioned this pull request Jan 6, 2022
Closed
5 tasks
@genuino2 genuino2 mentioned this pull request Feb 25, 2022
Closed
7 tasks
@rogersievers rogersievers mentioned this pull request Jun 22, 2022
Closed
5 tasks
@saqlainkhadim saqlainkhadim mentioned this pull request Oct 21, 2022
Closed
5 tasks
@rwkyyy rwkyyy mentioned this pull request Dec 9, 2022
Closed
2 tasks
@Estalhun Estalhun mentioned this pull request May 19, 2023
Closed
5 tasks
@camarfbi1 camarfbi1 mentioned this pull request Jul 5, 2023
Closed
5 tasks
@apeksha10 apeksha10 mentioned this pull request Aug 4, 2023
Closed
5 tasks
This was referenced Jan 12, 2024
Closed
Open
@vanushwashere vanushwashere mentioned this pull request Mar 6, 2024
Closed
5 tasks
@kingyyyyyy kingyyyyyy mentioned this pull request Apr 17, 2024
Open
5 tasks
@brettnzl brettnzl mentioned this pull request Apr 21, 2024
Open
5 tasks
@Hoogani Hoogani mentioned this pull request May 15, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@markjaquith @mikejolley