mirror of
https://github.com/zoriya/telescope.nvim.git
synced 2026-08-13 17:38:58 +00:00
fix(previewer): improve binary mime type check (#3083)
* fix(previewer): improve binary mime type check
Problem: mime type for a ts/js file can either return `text/plain` or
`application/javascript` based on the contents of the file.
Previously, this meant `application/javascript` would be considered
"possibly binary". This, in conjunction with how `vim.filetype.match`
does not give a result for a filename that ends in `.ts`, would lead to
a typescript file taking the path of `check_mime_type` and eventually
`mime_hook`.
Solution: Include `application/javascript` as a non-binary file type
during mime type check.
* [docgen] Update doc/telescope.txt
skip-checks: true
---------
Co-authored-by: Github Actions <actions@github>
(cherry picked from commit fac83a556e)
This commit is contained in:
@@ -192,9 +192,8 @@ previewers.file_maker = function(filepath, bufnr, opts)
|
||||
else
|
||||
if opts.preview.check_mime_type == true and has_file and opts.ft == "" then
|
||||
-- avoid SIGABRT in buffer previewer happening with utils.get_os_command_output
|
||||
local output = capture(string.format([[file --mime-type -b "%s"]], filepath))
|
||||
local mime_type = vim.split(output, "/")
|
||||
if mime_type[1] ~= "text" and mime_type[1] ~= "inode" and mime_type[2] ~= "json" then
|
||||
local mime_type = capture(string.format([[file --mime-type -b "%s"]], filepath))
|
||||
if putils.binary_mime_type(mime_type) then
|
||||
if type(opts.preview.mime_hook) == "function" then
|
||||
vim.schedule_wrap(opts.preview.mime_hook)(filepath, bufnr, opts)
|
||||
else
|
||||
|
||||
Reference in New Issue
Block a user