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

Stripe payments erroring because of invalid content-type #1006

Open
jdanna opened this issue Dec 4, 2023 · 2 comments
Open

Stripe payments erroring because of invalid content-type #1006

jdanna opened this issue Dec 4, 2023 · 2 comments
Assignees
Labels
Bug Integration An integration with a 3rd party solution or service
Milestone

Comments

@jdanna
Copy link

jdanna commented Dec 4, 2023

Expected Behavior

After adding stripe as a provider, attempting to pay an invoice results in an content-type error from stripe.

Current Behavior

Screenshots

IMG_3670
IMG_3671
IMG_3672

Stripe shows errors happened, but doesn’t provide any more detailed logs

Possible Solution

Looks like its related to how omnipay is making the request:

https://stackoverflow.com/questions/66241686/laravel-omnipay-stripe-unable-to-send-request-on-linux/66290005#66290005

I tried modifying the payment handler php to add the headers, but it didn’t work

Steps to Reproduce

-Enable online payments
-Enter stripe keys and enable stripe (i tested with both test mode and live mode keys in stripe - both had the same issue)
-Send an invoice
-Try to pay the invoice with a credit card

@nielsdrost7
Copy link
Contributor

nielsdrost7 commented Dec 4, 2023

Awesome bug report man! Thank you very much

$view_data doesn't have that extra header that the stackoverflow post is talking about
/application/modules/guest/controllers/Payment_information.php#L77

/application/modules/guest/controllers/Payment_information.php#L88

$('#ajax-card-form').load('<?php echo site_url('guest/payment_information/stripe');?>');

application/modules/guest/views/gateway/stripe.php

application/modules/guest/views/gateway/stripe.js

@nielsdrost7 nielsdrost7 self-assigned this Dec 10, 2023
@nielsdrost7 nielsdrost7 added this to the 1.6.2 milestone Dec 10, 2023
@nielsdrost7 nielsdrost7 pinned this issue Jan 19, 2024
@Verony-makesIT Verony-makesIT unpinned this issue Jan 30, 2024
@nielsdrost7 nielsdrost7 added the Bug label Mar 9, 2024
@naui95 naui95 added the Integration An integration with a 3rd party solution or service label Apr 26, 2024
@nielsdrost7
Copy link
Contributor

@jdanna great news! A pull-request is on its way to resolve this issue.
Once it's done we'll let you know

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Integration An integration with a 3rd party solution or service
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants