Create calibre-web-nginx.conf

[deldesir]

Allows access to /library/calibre-web//<title>/<webm_or_mp4_video_file>

[deldesir]

I symlink /library/calibre-web with /library/www/html/calibre-web

[holta]

Related:

• PR [Workaround for] OOM / memory "502 Bad Gateway" crashes w/ large videos' playback or download-to-client [SEE ALSO #37 & #53] #51

[holta]

Should this PR temporarily eliminate its 3 Ansible vars, hard-coding them to {/books, /libros, /libros} during testing of... ?

• PR [Workaround for] OOM / memory "502 Bad Gateway" crashes w/ large videos' playback or download-to-client [SEE ALSO #37 & #53] #51

[deldesir]

Yes. We don't need them for now though support to handle these hardcore variables in all Calibre-Web routes has been added in PR #33

[holta]

Yes. We don't need them for now though support to handle these hardcore variables in all Calibre-Web routes has been added in PR #33

Are you going ahead hard-coding them to {/books, /libros, /libros} in this PR, so people can test this?

[holta]

@deldesir if possible can you please explain more how this PR works, as a MM (memory management) workaround for Calibre-Web playback of larger videos?

[holta]

And can you confirm that the following PR is perhaps no longer necessary... ?

• PR [Workaround for] OOM / memory "502 Bad Gateway" crashes w/ large videos' playback or download-to-client [SEE ALSO #37 & #53] #51

While the following (interim?) PR is merged... !?

• PR Restrict NGINX (public?) access to "large" files only (e.g. .mp3, .mp4, .webm) in /library/calibre-web — for PR #51 OOM workaround #57

[deldesir]

This PR is where it all starts, with the creation of the calibre-web-nginx.conf file. Didn't know beforehand I would have to adjust it further until I was reported wide access was allowed to /library/calibre-web/ by @holta's comment (moved to issue #53)

• PR Fix OOM issue - affecting large videos' playback #51 is still relevant. We move away from loading file to memory to reading files directly using watchvideo.html (called in

  calibre-web/cps/web.py

  Lines 1568 to 1576 in fa70213

  	for fileExt in constants.EXTENSIONS_VIDEO:
  	if book_format.lower() == fileExt:
  	entries = calibre_db.get_filtered_book(book_id)
  	log.debug("Start video watching for %d", book_id)
  	video_path = os.path.join(config.config_calibre_dir, book.path)
  	video_file = glob.glob(os.path.join(video_path, "*." + book_format.lower()))
  	video_file = video_file[0]
  	video_type = "video/" + book_format.lower()
  	return render_title_template('watchvideo.html', videofile=video_file , videotype=video_type,

  )

• PR Deny wide access to entire calibre-web library directory #57 ensures only needed video file is readable for playback.