Magic Mage | Stripe Developers | Page 1

weak salmonBOT Apr 4, 2023, 7:03 PM

#

feral hamlet Apr 4, 2023, 7:07 PM

#

Ok, so what are you seeing happen? Are they any errors in your developer console you can share?

kind sequoia Apr 4, 2023, 7:17 PM

#

Nope, it just seems like it refuses to run the submit code in the js.

#

when I press on the button labeled Subscripe

#

the other "button" seems to work however.

feral hamlet Apr 4, 2023, 7:20 PM

#

Are you sure you're triggering a form submit?

#

Your subscribeBtn element is just an a, not a button

#

so might not be triggering the form submit youre expecting

#

Instead you could add an event listener for click on the a/subscribeBtn

kind sequoia Apr 4, 2023, 7:30 PM

#

hmm or this might be the only way:

feral hamlet Apr 4, 2023, 7:31 PM

#

That's another option, sure

kind sequoia Apr 4, 2023, 7:49 PM

#

Welp I have committed that fix now: now to debug it.

kind sequoia Apr 4, 2023, 8:17 PM

#

#

oh cool

#

it might be the redirect url

#

as I want to make it redirect to the homepage.

#

from that to https://localhost:44372/

feral hamlet Apr 4, 2023, 8:19 PM

#

You should be able to set the return_url as you need it

kind sequoia Apr 4, 2023, 8:20 PM

#

yeah, I tried that, but it seems to not like the fact that I set it to '/'

feral hamlet Apr 4, 2023, 8:20 PM

#

Can you show your code?

kind sequoia Apr 4, 2023, 8:23 PM

#

sure, it should be on the server side part of it.

feral hamlet Apr 4, 2023, 8:28 PM

#

Oh, right: return_url: "/"

#

You need a valid complete url there

#

not a relative path

#

return_url: "https://example.com/"

kind sequoia Apr 4, 2023, 8:31 PM

#

Is it ok if I leave in the just the price id's and the product ids? I scrubbed from it the api keys and secrets though.

#

https://gist.github.com/AraHaan/ac2a44384d18f63713875c3119a3ff55

kind sequoia Apr 4, 2023, 8:33 PM

#

feral hamlet You need a valid complete url there

so there is no way of getting the actual base url then in case of localhost / on the college issued webserver?

#

oh wait it was in the js part of the application.

feral hamlet Apr 4, 2023, 8:34 PM

#

Well you could control that in your code based on an environment variable etc

#

if live mode, use real URL, if test mode use local host

kind sequoia Apr 4, 2023, 8:36 PM

#

I mean that could help with the possibility that the debug build will need to be pushed to the webserver now because the account I made for it only allows for test mode only (plus the project is technically in testing right now anyways there).

feral hamlet Apr 4, 2023, 8:39 PM

#

Since its client side code you can also look at the URL/hostname and reflect that

kind sequoia Apr 4, 2023, 8:39 PM

#

hmm could

#

not sure if window.location.origin could help with this 🤔 especially when the "live" page is something like http://dotnet.<college name>.edu/<capstone project name>/ (yes they do not have https enabled for our pages yet).

feral hamlet Apr 4, 2023, 8:50 PM

#

Well you're going to need https to use this in live mode

#

https://stripe.com/docs/security#https-hsts-secure-connections

kind sequoia Apr 4, 2023, 8:52 PM

#

yeah, hence why I would need use it in test mode there too with debug builds only so we got to push the debug builds or it there too.

atomic forge Apr 4, 2023, 8:55 PM

#

Hi there 👋 taking over, as my colleague needs to step away

It sounds like the resolution is: you must find a way for your local host to serve over HTTPS. Your live mode website would also need to server over HTTPS. Are there any outstanding questions that I'm missing?

kind sequoia Apr 4, 2023, 8:56 PM

#

I will ask my instructor if they can enable https for us.

atomic forge Apr 4, 2023, 9:00 PM

#

Apologies, I forgot that HTTP is okay for test mode (instead of HTTPS), but you would still need to not use a relative URL, so you need some mechanism or system that exposes your localhost to the internet and has a non-relative URL

kind sequoia Apr 4, 2023, 9:03 PM

#

        // handle webserver version and localhost version.
        const url = (window.location.hostname == '127.0.0.1' || window.location.hostname == 'localhost')
            ? window.location.origin + '/' : window.location.origin + '/<project name>/';

I have modified the js code to basically assign the url to a variable and then have the submit event use said variable when doing the payment confirmation.

#

it worked lol

#

nice

atomic forge Apr 4, 2023, 9:04 PM

#

Right on!

kind sequoia Apr 4, 2023, 9:06 PM

#

Also I tested with the 42 card.

#

oh no

#

it keeps failing to post to the webhook.

#

🤔

#

But now I know that there is only about a few events to worry about customer.subscription.updated, customer.subscription.deleted, customer.subscription.paused, and customer.subscription.resumed the created one can technically be ignored but I think there are cases where it is instantly in paid status?

atomic forge Apr 4, 2023, 9:12 PM

#

Yeah, you could listen for invoice.paid in that case

kind sequoia Apr 4, 2023, 9:18 PM

#

yeah, I guess so.

#

But now need to figure out why the webhook is failing locally with Failed to POST: Post "http://localhost:44372/webhook": read tcp [::1]:13507->[::1]:44372: wsarecv: An existing connection was forcibly closed by the remote host.

#

I guess I could skip the webhook though and on the root page redirect have it check the subscription by fetching it like normal and then if it's in active status manually update the database.

#

since I do have my ERD with email and name for customers so it's easy to fetch the stripe customer object for them without the stripe customer id, then fetch their subscription and then it should be ok.

atomic forge Apr 4, 2023, 9:25 PM

#

I suspect that's because Stripe doesn't know where "http://localhost:44372/webhook" is, right? Like, the request has to go through DNS and there's no way to associate your localhost with an IP address or hosted URL.

So yeah, you could rely on the redirect in this case, or use a domain URL or IP address that's specific to your machine in order to expose it to the internet

kind sequoia Apr 4, 2023, 9:27 PM

#

yeah I guess

#

perhaps it might be the cli itself needing to convert localhost to an ip address however it might be a problem because our ISP issues only ipv6 now.

#

or so it seems so that is

#

Yeah, actually I think it’s best to skip webhooks and actually just update the db records by fetching the subscription when the payment redirects to the home index page on the site.

#

It would also make it simpler as well too.

atomic forge Apr 4, 2023, 9:38 PM

#

If this is eventually going to go into production, I would recommend not relying on the redirect to perform business logic, and instead focus on fixing the webhooks issue, since redirects can be sloppy (e.g. what happens if the user closes the page or the redirect fails?)

kind sequoia Apr 4, 2023, 9:50 PM

#

true

#

but even then the next time they visit the site it would still be able to grab the subscription data.

atomic forge Apr 4, 2023, 9:55 PM

#

I'm not sure I follow

kind sequoia Apr 4, 2023, 10:03 PM

#

#

Basically something like this.

atomic forge Apr 4, 2023, 10:05 PM

#

I still don't understand what the question is. Sorry, I'm trying to infer where the issue/question is in your messages, but I can't.

kind sequoia Apr 4, 2023, 10:06 PM

#

Basically for some reason the webhook seems to not be working so I guess doing this could be a good fallback though.

atomic forge Apr 4, 2023, 10:09 PM

#

Right, but that's because Stripe doesn't know where "http://localhost:44372/webhook" is, right? The request has to go through DNS and there's no way for it to associate your localhost with an IP address or hosted URL. You'd have to use a URL the DNS can lookup and route the request to

kind sequoia Apr 4, 2023, 10:09 PM

#

yeah

#Magic Mage