kostenko | Stripe Developers | Page 1

bright yarrowBOT Nov 9, 2022, 11:24 AM

#

brittle notch Nov 9, 2022, 11:25 AM

#

👋 happy to help

#

I'm not sure that 70MB of Ram is huge per se

#

If you really feel that we should improve on that I would strongly encourage you to write to support https://support.stripe.com/?contact=true and give us as much information as possible so our internal team might be able to investigate further and see if some improvements could be made

Stripe: Help & Support

Find help and support for Stripe. Our support center provides answers on all types of situations, including account information, charges and refunds, and subscriptions information. Get your questions answered and find international support for Stripe.

obsidian gust Nov 9, 2022, 11:28 AM

#

I think it is huge for invisible iframe of a random tab of a random website. Our full-featured react app with material UI uses less

brittle notch Nov 9, 2022, 11:30 AM

#

Invisible? what are you testing exactly?

obsidian gust Nov 9, 2022, 11:32 AM

#

I've included stripe.js on every page as was requested in API. We display payment widgets only on specific upgrade page, on other pages we don't display any stripe elements

brittle notch Nov 9, 2022, 11:33 AM

#

you shouldn't include Stripe elements everywhere

#

I've included stripe.js on every page as was requested in API
do you mind sharing where does it say so?

obsidian gust Nov 9, 2022, 11:34 AM

#

https://stripe.com/docs/js/including

Stripe JS Reference

#

To best leverage Stripe’s advanced fraud functionality, include this script on every page, not just the checkout page. This allows Stripe to detect suspicious behavior that may be indicative of fraud as customers browse your website.

sour pecan Nov 9, 2022, 11:36 AM

#

Hey! Taking over for my colleague. Let me catch up.

#

Did you tried to load the asynchronously and deferred loading of Stripe.js ?

obsidian gust Nov 9, 2022, 11:40 AM

#

do you mean calling loadStripe() instead of including the script directly in <head>?

sour pecan Nov 9, 2022, 11:50 AM

#

You can either load Stripe.js as module which is the easiest way to asynchronously do that. Or load Stripe.js using the async or defer attribute on the script tag.
https://stripe.com/docs/js/including#:~:text=Asynchronous and deferred loading of Stripe.js

obsidian gust Nov 9, 2022, 11:52 AM

#

one moment, let me try, but I'm not sure it changes anything - it is about load order (before or after dom)

obsidian gust Nov 9, 2022, 12:23 PM

#

same thing, 70 megabytes

#

there are m-outer-...*.html 60+ megabytes and 5.5 Mb inner.html

sour pecan Nov 9, 2022, 12:33 PM

#

Why are you considering 70MB as a "memory leak" ?

obsidian gust Nov 9, 2022, 12:37 PM

#

It is a guess and probably it is up to Stripe to debug. But I consider 70 megabyte RAM as wasting our clients' memory

sour pecan Nov 9, 2022, 12:43 PM

#

Do you have any references that consider 70MB is a leak and Waste ? not sure about that, but if you have some recommendation we are open to discuss about it

obsidian gust Nov 9, 2022, 12:49 PM

#

If you press "Shift+Esc" on chrome, you can see tab memory usage. Average amount is around 30 megabytes. Except of YouTube, Jira and Slack

edgy dawn Nov 9, 2022, 12:50 PM

#

This is not something we can help with directly. I'd recommend either filing an issue over on the GH repo (https://github.com/stripe/stripe-js), or writing in to support

obsidian gust Nov 9, 2022, 12:54 PM

#

Okay, thank you. I wrote to support, this repo is just a repo of script loader, does not include any stripe.js implementation

edgy dawn Nov 9, 2022, 12:55 PM

#

Sure, the actual Stripe.js source is an internal repository. But that would be where I'd recommend filing the issue as the eng team who works on it can take a look

#kostenko