-
Notifications
You must be signed in to change notification settings - Fork 37
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
Please check your URL
YouTube bug in production
#50
Comments
I have a staging server on the same stack as production, and it's working fine. I'm wondering if this is some rate limiting somehow. Any ideas on how I can correct this? |
I’d recommend doing a composer update to get the latest embed/embed version to be some a few fixes recently but rate limiting is common and is why I added caching feature in the settings, which I would recommend trying. I have attached my discord to my profile recently if you need more direct help debugging also. |
Seeing the same issue on 1.3.4. Done composer update for embed/embed with no luck. |
I added |
@Moorst your issue could be related to rate limiting. Are you using the cache feature? If not try using that to prevent the re-hitting of the Youtube servers. Else you might be able to check your logs and if you like to discuss your logs more privately I'm on Discord and you can find my discord via my profile. |
Might be due to rate limit as they're now showing fine. I did try putting the cache on but got an internal server error, will have to debug this locally. |
@Moorst Youtube has been rate limiting alot lately but I think I need to look at a feature here to generate a snapshot of the embed object on "save" of an entry and then if it fails during the rendering to use that version. It'll mean that we can still access data if the rate limiting happens and might solve generate issues with the 5xx error due to server outage or server errors. |
@reganlawton |
@reganlawton - so I was looking into how the field stores data, and this is what it stores in the database:
So why not just, at save time, actually populate the |
@adrienne this is what I discussed in my comment 6 days ago, to create a snapshot on the field on create. |
@reganlawton - yeah, i'm just suggesting that you even already have a sort of hook to hang that on, in the data you're already saving. |
Having this problem with v1.3.6, I've done a composer update and checked embed/embed dependency. |
If I try to turn on caching I get Is the rate limit based on how many times the API is hit with the code from this plugin? Does that mean as you get more users this will happen more frequently? I have used Embedded Assets by spicy web for other sites but I really liked the admin video preview that comes with this. |
Hi there! First, thanks so much for the wonderful
oEmbed
plugin. I'm seeing a current issue on my Laravel Forge / Digital Ocean production environment. It only happens with valid YouTube URLs:The output is simply
Please check your URL.
Here is an example YouTube URL that I'm trying:https://www.youtube.com/watch?v=zVi_9rlepbE
I tested it in my Laravel Valet local environment and I couldn't encounter the same issue. Thanks so much for your help!
The text was updated successfully, but these errors were encountered: