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

Not using v.zero SDK #14

Closed
ayb opened this issue Feb 8, 2016 · 1 comment · Fixed by #22
Closed

Not using v.zero SDK #14

ayb opened this issue Feb 8, 2016 · 1 comment · Fixed by #22

Comments

@ayb
Copy link

ayb commented Feb 8, 2016

Current implementation is posting credit card data directly back to the server

Started PATCH "/checkout/update/payment" for 127.0.0.1 at 2016-02-08 05:56:54 -0500
Processing by Spree::CheckoutController#update as HTML
Parameters: {"utf8"=>"✓",
"authenticity_token"=>"KLjo/1QEPRso8DOuTXlcCVZPlDD3/BcIxuWTUzh+/HhpbDw0gCfDVnYRiCHK8u00oRJGxW0Mbx33o6B0AIjmgg==",
"order"=>{"payments_attributes"=>[{"payment_method_id"=>"2"}], "coupon_code"=>""}, "payment_source"=>{"2"=>{"name"=>"Test
User", "number"=>"[FILTERED]", "expiry"=>"11 / 17", "verification_value"=>"[FILTERED]",
"address_attributes"=>{"firstname"=>"Test ", "lastname"=>"User", "company"=>"", "address1"=>"431 Test Ave", "address2"=>"",
"city"=>"Testing", "country_id"=>"232", "state_id"=>"3561", "state_name"=>"", "zipcode"=>"10001", "phone"=>"212-111-1111",
"alternative_phone"=>""}, "cc_type"=>"visa"}}, "state"=>"payment"}

v.zero SDK does not post back credit card data, just a payment method nonce:

{"payment_method_nonce"=>"761e1fad-94a3-4737-add1-6cccf7ec1b8d"}

@mike-bourgeous
Copy link

Related to this request for client-side submission to Braintree:

I work on a large ecommerce site that is currently planning a move from Spree to Solidus. We use client-side encryption and tokenization on both checkout and admin. If we reimplement this for Solidus based on this gem, would these changes be useful/welcome here?

This was referenced Mar 31, 2016
gsmendoza pushed a commit to gsmendoza/solidus_braintree that referenced this issue Dec 28, 2022
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 a pull request may close this issue.

2 participants