<p>pdf.js bergantung pada url untuk memuat ekstensi 'pdf'</p>

Dibuat pada 12 Feb 2014  ·  13Komentar  ·  Sumber: mozilla/pdf.js

Jika server tidak menyediakan header Content-Disposition, pdf.js bergantung pada url untuk memuat ekstensi 'pdf'. Tapi URL adalah pencari lokasi, bukan nama.
Langkah-langkah untuk mereproduksi:

mv web/compressed.tracemonkey-pldi-09.pdf web/compressed.tracemonkey-pldi-09
sed -i 's/compressed.tracemonkey-pldi-09.pdf/compressed.tracemonkey-pldi-09/g' web/viewer.js
firefox web/viewer.html

Sekarang klik unduh. Anda akan ditawari file 'document.pdf'. Nama itu harus lebih bermakna.
Bug juga terjadi ketika saya meninggalkan ekstensi pdf di server web apache saya.

Solusi yang diusulkan:
Gunakan judul pdf. (sebagai kode viewer.js ini). Judul ini juga digunakan oleh firefox untuk fungsi File -> "simpan halaman sebagai" saat menampilkan halaman HTML seperti http://en.wikipedia.org/wiki/Internet_media_type .

Tidak setiap halaman web html diakhiri dengan .html. Alih-alih oleh ekstensi, tipe dokumen ditentukan oleh tipe MIME-nya.
Namun, kebanyakan file pdf memiliki ekstensi pdf, dan kebanyakan pdf online juga memiliki nama yang bagus untuk disimpan di url.
Saya tidak tahu apakah metode pengambilan baru harus menimpa pengambilan url, atau menjadi fallback untuk itu.

Lihat juga # 3455.

1-core 5-good-beginner-bug

Semua 13 komentar

@tokopedia

Ada pembaruan tentang ini? Sudah terbuka selama lebih dari dua tahun sekarang. Karena parameter file saya adalah panggilan server yang mengirim kembali file pdf, penampil pdf tidak dapat mendeteksi nama file karena tampaknya mencari ekstensi .pdf sehingga saya terjebak dengan "document.pdf "saat mengunduh dan" untitled.pdf "di bilah jendela saat melihat.

Akan berguna jika kita juga dapat menentukan "judul" di URI serta "file" seperti ... / pdf-viewer / viewer.html? File = "..." & title = "... "

Saya tahu bahwa saat ini pekerjaan sedang dilakukan di # 7554 untuk mendukung header Content-Disposition , yang merupakan cara untuk mengatasi masalah ini. Saya setuju, bagaimanapun, bahwa document.pdf bukanlah nama yang terbaik dan kita mungkin perlu memperbaiki fungsi untuk mendapatkan nama (file). Tambalan untuk ini dipersilakan, jadi saya memberi label ini sebagai bug pemula yang baik karena seharusnya tidak terlalu sulit untuk diterapkan.

@timvandermeij Luar biasa terima kasih, saya yakin mendukung Content-Disposition akan benar-benar memperbaiki masalah saya.

Saya setuju, karena saya sedang membaca kode saya perhatikan seharusnya tidak terlalu sulit untuk hanya menambahkan parameter URL lain untuk nama file. Saya akan mencobanya dalam beberapa hari ke depan, Terima kasih.

Tambalan untuk ini dipersilakan, jadi saya memberi label ini sebagai bug pemula yang baik karena seharusnya tidak terlalu sulit untuk diterapkan.

@timvandermeij Harap diingat bahwa dalam PR # 4956 kami sengaja menjauh dari membiarkan berbagai parameter hash mempengaruhi penampil (kecuali debugging diaktifkan, lihat https://github.com/mozilla/pdf.js/wiki/Debugging-pdf.js) .
Oleh karena itu saya tidak berpikir bahwa kita harus memungkinkan untuk menentukan title menggunakan parameter hash!

Terutama mengingat itu akan menjadi non-standar (dalam konteks http://www.adobe.com/content/dam/Adobe/en/devnet/acrobat/pdfs/pdf_open_parameters.pdf), dan dibandingkan dengan Content-Disposition mendekati PR # 7554, itu benar-benar tidak menambah banyak nilai.

Maaf, saya seharusnya lebih jelas. Maksud saya, tambalan dipersilakan untuk meningkatkan fungsi yang menentukan nama file dari URL. Saya rasa kita bisa melakukan lebih baik di sana daripada hanya mengandalkan ekstensi file. Saya setuju bahwa kita tidak boleh menambahkan lebih banyak parameter hash.

Apa status masalah ini? Apakah ini masih buka?

Apa status masalah ini? Apakah ini masih buka?

@anirudhrb masih dibuka, ada upaya untuk mengimplementasikannya di # 7554, maukah Anda berkontribusi untuk itu?

@yurydelendik Ya, saya ingin berkontribusi. Apa yang diharapkan dari PR untuk masalah ini?

@anirudhrb , Anda dapat menggunakan PR di atas sebagai basis karena memiliki remoting data yang agak benar - kami mengharapkan patch kecil dengan tes unit. Kami tidak memerlukan penguraian Disposisi-Konten spesifikasi, tetapi cukup untuk mendapatkan nama file.

@yurydelendik Saya sudah mulai mengerjakan ini. Ini adalah upaya pertama saya untuk berkontribusi pada proyek sumber terbuka. Saya perlu waktu untuk terbiasa dengan basis kode. :)

@yurydelendik , @timvandermeij Bisakah saya menangani masalah ini jika Anda semua

Ada permintaan tarik di atas yang terlihat seperti arah yang benar, tetapi sudah tidak ada aktivitas untuk itu. Jika Anda tertarik untuk memperbaikinya, kedengarannya bagus. Saya akan bertanya apakah penulis asli masih berencana untuk mengerjakannya.

Diperbaiki di # 9379.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat