Eric Portis

Should Critical-CH restarts be limited to idempotent requests (or even just GET)?

@yawuxi Thank you so much for both the PR, and the ping. Your description of the problem/solution makes sense but I have not made the time to do an actual...

@yawuxi I finally (!) had some time to try to wrap my brain around this today. TL;DR I am still not sure what the best course of action is and...

I'm able to reproduce the basic case (`playerRef.current` is undefined within a `useEffect`). If you can, @Figumari or @kostas-pblworks, could you share an example of exactly how you are trying/failing...

@jlooper-cloudinary Yes, but I'm still not clear on the use case for needing a ref before the element exists. See my [comments on the PR](https://github.com/cloudinary-community/next-cloudinary/pull/598#issuecomment-3535175270).

We may inherit this problem from upstream, at least from https://github.com/cloudinary-community/cloudinary-util/tree/main/packages/url-loader and maybe from https://github.com/cloudinary/js-url-gen. ``` import { constructCloudinaryUrl } from '@cloudinary-util/url-loader'; const url = constructCloudinaryUrl({ options: { src: 'sample',...

Ah, URLs don't 400 if the two commas are at the end of the text because it only results in an empty transformation component. If the two commas are in...

js-url-gen encodes the two commas correctly ``` import {Cloudinary} from '@cloudinary/url-gen'; const cld = new Cloudinary({ cloud: { cloudName: 'eric-cloudinary' } }); const myImage = cld.image('sample'); import {source} from "@cloudinary/url-gen/actions/overlay";...

Testing this with @yawuxi ’s original reduced example in #587, I still end up with duplicate iframes. Digging into why...

@Vaibhav91one None of these options is quite what I had in mind, but I do like them. I need to think this through, but unfortunately the time for Hacktoberfest reviews...