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.

Sign up for GitHub

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

Jump to bottom

access to video.min.js.map is forbidden #3106

Closed
frontendschlampe opened this issue Feb 12, 2016 · 9 comments
Closed

access to video.min.js.map is forbidden #3106

frontendschlampe opened this issue Feb 12, 2016 · 9 comments
Assignees
@gkatsev
Labels
bug confirmed

Comments

@frontendschlampe
Copy link

[Error] Failed to load resource: the server responded with a status of 403 (Forbidden) https://vjs.zencdn.net/5.4.6/video.min.js.map

see ticket #700

OsX El Captain
Safari 9.0.3

http://stackoverflow.com/questions/18407543/video-js-map-throwing-a-404-not-found
@frontendschlampe frontendschlampe changed the title access to video.min.js.map is vorbidden access to video.min.js.map is forbidden Feb 12, 2016
@gkatsev
Copy link
Member

gkatsev commented Feb 12, 2016

hm... I thought we got that taken care of.

@tjbee
Copy link

tjbee commented Apr 12, 2016

Has this been resolved?

@gkatsev
Copy link
Member

gkatsev commented Jul 26, 2016

Nope, unfortunately, still an issue.

@gkatsev gkatsev self-assigned this Jul 26, 2016
@gkatsev gkatsev mentioned this issue Aug 16, 2016
Closed
8 tasks
@gkatsev
Copy link
Member

gkatsev commented Aug 17, 2016

Will be fixed once videojs 3.12 is released as part of #3445. Closing.

@gkatsev gkatsev closed this as completed Aug 17, 2016
@jemoreno
Copy link

This is still an issue

@brandonocasey brandonocasey reopened this Oct 14, 2016
@gkatsev
Copy link
Member

gkatsev commented Oct 14, 2016

What version of videojs is it still an issue for?

@jemoreno
Copy link

http://vjs.zencdn.net/5.8/video.js.map
http://vjs.zencdn.net/5.9/video.js.map
http://vjs.zencdn.net/5.10/video.js.map
http://vjs.zencdn.net/5.11/video.js.map

They all return a 403 error

@jemoreno
Copy link

any update on this?

@gkatsev gkatsev mentioned this issue Oct 24, 2016
Merged
@gkatsev
Copy link
Member

gkatsev commented Oct 24, 2016

This will be removed from 5.11 soon (#3710) and already isn't available in 5.12. We'll consider adding proper sourcemaps sometime later.

@gkatsev gkatsev closed this as completed Oct 24, 2016
@gkatsev gkatsev mentioned this issue Oct 24, 2016
Closed
gkatsev added a commit that referenced this issue Oct 24, 2016
@gkatsev
fix: remove sourcemaps from the output
31b9d2c 
Source Maps, while nice, aren't perfect and the ones we were outputting
weren't right.

Fixes #3701, #3106.
gkatsev added a commit that referenced this issue Oct 25, 2016
@gkatsev
fix: remove sourcemaps from the output (#3710)
9f809cf 
Source Maps, while nice, aren't perfect and the ones we were outputting
weren't right.

Fixes #3701, #3106.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@gkatsev gkatsev

Labels
bug confirmed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

new build that outs source es6 files into es5 folder gkatsev/video.js
5 participants
@gkatsev @frontendschlampe @brandonocasey @jemoreno @tjbee