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

Fix/494 #537

Merged
merged 2 commits into from
Mar 21, 2017
Merged

Fix/494 #537

merged 2 commits into from
Mar 21, 2017

Conversation

gorjanz
Copy link
Contributor

@gorjanz gorjanz commented Mar 21, 2017

In order to address #494 and #530

two fixes are introduced with this pull request:

  1. as suggested by @devmaster72 the auto-start of the video is fixed, by taking into the account the 'paused' prop passed
  2. An IOException was constantly being thrown from the android Media player classes, because instead of only a URL, the media player, for a dataSource was expecting a whole ContentProvider initialised with context

Gorjan Zajkovski added 2 commits March 21, 2017 14:25
… thrown in android.MediaPlayer, due to expected uri resource with content provider instead of only a URL
@ghost
Copy link

ghost commented Mar 21, 2017

thank you @gorjanz for the PR

@mattapperson mattapperson merged commit 9ce1f4b into TheWidlarzGroup:master Mar 21, 2017
mattvot pushed a commit to calvium/react-native-video that referenced this pull request May 12, 2017
* master:
  Added flag for background mode (TheWidlarzGroup#563)
  issue 310 - Added a new property, ignoreSilentSwitch.  (TheWidlarzGroup#403)
  Add progressUpdateInterval to android-exoplayer (TheWidlarzGroup#540)
  Initial WPF Support (TheWidlarzGroup#385)
  added progressUpdateInterval to Android (TheWidlarzGroup#512)
  load new source if it is different (TheWidlarzGroup#502)
  Pausing progress if it was playing and the stating again (TheWidlarzGroup#526)
  bump android dependencies (TheWidlarzGroup#524)
  Update ExoPlayer to r2.2.0 (TheWidlarzGroup#505)
  Update readme (TheWidlarzGroup#497)
  [Android] App crash when unmounting video (TheWidlarzGroup#533)
  Fix/494 (TheWidlarzGroup#537)
  fix(RCTVideo.m): Fixed status bar disappearing issue
@karltaylor
Copy link
Contributor

karltaylor commented Nov 17, 2017

@gorjanz Can you remember why you removed headers from

setDataSource(mThemedReactContext, parsedUrl, headers);`

This stops cookies being sent on android.

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