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

Added host-option to embed youtube from 'nocookie'-domain #557

Closed
wants to merge 2 commits into from

Conversation

AlexandreKilian
Copy link

Fix for #272 to allow changing the iFrame Domain to "nocookie".
Linted, tested, built and demo updated.

@Betree
Copy link

Betree commented Jan 24, 2019

This is awesome @AlexandreKilian! Thanks for your work on this. Will definitely be useful for us.

@cookpete cookpete closed this in 0fbb0c4 Jan 24, 2019
@cookpete
Copy link
Owner

cookpete commented Jan 24, 2019

I really didn't want to add a new top level host prop for a Youtube-specific edge case.

In 1.9.0 I've added embedOptions to the youtube config that gets spread here after the other options. Then you can do:

<ReactPlayer
  url='...'
  config={{ 
    youtube: { 
      embedOptions: { 
        host: 'https://www.youtube-nocookie.com' 
      } 
    } 
  }}
/>

And override any other options in future, if you want to.

cookpete added a commit that referenced this pull request May 23, 2020
Fixes #896
Thoroughly fixes #272
Thoroughly fixes #557
albanqoku added a commit to albanqoku/react-player that referenced this pull request Feb 24, 2021
Closes cookpete/react-player#557
Allows the use of the `host` option, which fixes cookpete/react-player#272
Webmaster1116 added a commit to Webmaster1116/video-player that referenced this pull request May 20, 2021
Closes cookpete/react-player#557
Allows the use of the `host` option, which fixes cookpete/react-player#272
Webmaster1116 added a commit to Webmaster1116/video-player that referenced this pull request May 20, 2021
webmiraclepro added a commit to webmiraclepro/video-player that referenced this pull request Sep 9, 2022
Closes cookpete/react-player#557
Allows the use of the `host` option, which fixes cookpete/react-player#272
webmiraclepro added a commit to webmiraclepro/video-player that referenced this pull request Sep 9, 2022
philip-luther added a commit to philip-luther/react-player that referenced this pull request Nov 22, 2024
Closes cookpete/react-player#557
Allows the use of the `host` option, which fixes cookpete/react-player#272
philip-luther added a commit to philip-luther/react-player that referenced this pull request Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants