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

Maximum call stack size exceeded with video.js 6.7.2 #208

Closed
zeusworkspace opened this issue Feb 19, 2018 · 9 comments
Closed

Maximum call stack size exceeded with video.js 6.7.2 #208

zeusworkspace opened this issue Feb 19, 2018 · 9 comments
Labels

Comments

@zeusworkspace
Copy link

I getting this error on console when the player is loaded.

screenshot from 2018-02-19 12-43-30

@thijstriemstra
Copy link
Member

@devaptas what browser? what version of videojs-record, video.js? What does your player config look like? How am I supposed to debug this without any additional information?

@zeusworkspace
Copy link
Author

zeusworkspace commented Feb 19, 2018

Chrome Version 64.0.3282.167, viodejs-record 2.1.0, video.js 6.7.2, follow the config:

    var player, mediaOptions;

    if(isVideo) {
        var player = videojs('componentRecord',
        {
            //video.js options
            controls: true,
            width: 600,
            height: 300,
            plugins: {
                // videojs-record plugin options
                record: {
                    audio: true,
                    video: true,
                    maxLength: 1800,
                    debug: true
                }
            }
        });
        
        mediaOptions = {           
            tag: 'video',
            type: 'video/webm',
            ext: '.mp4',
            gUM: {video: true, audio: true}            
        };
    } else {
        var player = videojs('componentRecord',
        {
            controls: true,
            width: 600,
            height: 300,
            language: 'pt-BR',
            plugins: {
                wavesurfer: {
                    src: 'live',
                    waveColor: 'black',
                    progressColor: '#fff',
                    debug: true,
                    cursorWidth: 1,
                    msDisplayMax: 20,
                    hideScrollbar: true
                },
                record: {
                    audio: true,
                    video: false,
                    maxLength: 1800,                    
                    debug: true
                }
            }
        });
        
        mediaOptions = {
            tag: 'audio',
            type: 'audio/mp3',
            ext: '.mp3',
            gUM: {audio: true}            
        };
    }
    
    /* -------------  Trigger the vjs-device-button button ----------------- */
    player.on('ready', function(){
        player.recorder.getDevice();
    });

    player.on('deviceReady', function(){
        $('.vjs-record-button').attr('title', '". Yii::t('app', 'Start Record')."');

        // Waveform has to play just when recording
        if(!isVideo)
            player.waveform.pause();
    });

@thijstriemstra
Copy link
Member

@devaptas I think video.js 6.7.2 is the issue, use 6.6.3 instead. I'll take a look at 6.7.2 support.

@thijstriemstra thijstriemstra changed the title Maximum call stack size exceeded Maximum call stack size exceeded with video.js 6.7.2 Feb 19, 2018
@zeusworkspace
Copy link
Author

zeusworkspace commented Feb 19, 2018

solved, tnks. So, I think the best solution is lock the video.js version on package.json, bower.json. I'm using the Asset Packagist to download the packages and it auto-check and download dependencies from this files.

@thijstriemstra
Copy link
Member

@devaptas finding the issue, fixing it and putting out a new release should be probably just as fast. Also note that video.js >= 6.7.0 are marked as pre-releases..

@thijstriemstra
Copy link
Member

And @devaptas this is probably only happening for audio-only?

@zeusworkspace
Copy link
Author

both

@thijstriemstra
Copy link
Member

thijstriemstra commented Feb 19, 2018

Here's the diff for changes in 6.7: videojs/video.js@v6.6.3...v6.7.2

The bug also exists in videojs-wavesurfer (and should be fixed first): collab-project/videojs-wavesurfer#49

@thijstriemstra
Copy link
Member

Fixed with 869eb86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants