Sentry-javascript: Tambahkan teks ke elemen di remah roti ui.click

Jadi, ini adalah sesuatu yang ingin saya lakukan, tetapi saya khawatir tentang pembengkakan fungsi serialisasi DOM kami ke titik yang mulai menambahkan overhead yang nyata ke penangan klik.

Tapi, saya akan menjelajah.

Salah satu alternatif yang mudah adalah dengan melakukan

target = elem.outerHTML;

yang akan menampilkan elemen yang diklik persis seperti apa adanya. Anda akan kehilangan struktur pohon - tetapi dalam pengalaman pribadi saya itu tidak banyak membantu seperti melihat elemen penuh dan aktual yang biasa saya lihat dalam kode saya sepanjang waktu dan karena itu lebih cepat dapat mengenali.

Itu akan menyelesaikan https://github.com/getsentry/raven-js/issues/576 juga

@soroushhakami – kami mulai dengan outerHTML terlebih dahulu tetapi tidak digunakan lagi untuk metode utilitas ini.

Alasan kami tidak memilih outerHTML (seperti yang dikomentari @mitsuhiko ):

• format pemilih CSS dapat menampilkan lebih banyak informasi dengan lebih sedikit karakter
• outerHTML dapat menyertakan data sementara yang berisik, mis
  
  
  
  • string data-reactid (di Bereaksi 0.14 dan sebelumnya)
  
  
  • atribut title dihasilkan secara dinamis yang ditujukan untuk tooltips
  
  
• kita akan mendapatkan banyak elemen kosong addEventListener dikonfigurasi
• kami memiliki tombol dengan kelas/konten yang sama, yang hanya dikontekstualisasikan melalui informasi leluhur

Pada dasarnya, outerHTML tampak bagus ketika Anda menunjukkan banyak contoh hipotetis yang terlihat bagus, tetapi pengalaman kami menjalankannya secara langsung selama beberapa minggu adalah bahwa itu tidak cukup.

Ah begitu, masuk akal, terima kasih atas penjelasannya! 👍

Saya telah menambahkan tag ID ke semua tombol saya. Terkejut bahwa mereka tidak muncul di remah roti. Bahkan jika itu adalah tag khusus, saya akan senang dengan itu. Tapi ID tag tampaknya ideal karena keunikannya.

@nblasgen - jika Anda dapat memberikan kasus uji yang gagal, saya akan senang melihat apa yang terjadi.

Tapi sekarang kode seharusnya menangkap ID , dan ada tes yang memverifikasi ini .

+1 untuk menampilkan ID, atau tag khusus seperti data-sentry-id="Some text here"

Saya akan menutup yang ini untuk saat ini, karena kami menyadari bagian yang hilang ini dan kami akan meninjau kembali bagaimana hal ini bekerja di versi utama berikutnya.

Ref: https://github.com/getsentry/raven-js/issues/783
Ref: https://github.com/getsentry/raven-js/issues/576

Hanya ingin menambahkan - Saya menggunakan react & styled-components sehingga nama kelas saya dibuat secara otomatis, akan sangat bagus untuk memiliki lebih banyak konteks (tanpa harus secara manual captureBreadcrumb ):

@kamilogorek ada pembaruan tentang ini?

Apakah ada kemungkinan untuk menyempurnakan klik "pesan"?

Mungkin, lebih banyak data dapat diteruskan ke breadcrumbCallback agar orang dapat memilih apa yang mereka butuhkan tanpa mengirimkannya ke penjaga secara default? Mungkin bahkan seluruh elemen dom?

@TuxujPes itu mungkin terjadi di rilis utama berikutnya, namun tidak dalam rilis ini, karena memerlukan beberapa perubahan API publik yang hanya dapat kami perkenalkan di rilis utama.

@kamilogorek kedengarannya bagus. Adakah perkiraan tanggal rilis utama?

Di suatu tempat di dekat akhir Q3 (permisi waktu respons yang lama, sedang berlibur).

+1 untuk menampilkan ID, atau tag khusus seperti data-sentry-id="Some text here"

Mereka bisa menganalisis jika ada informasi ini dan membawanya. Jika ada, biarkan pemilih dengan tombol untuk informasi lebih lanjut.

@hiagodotme @TuxujPes @adamreisnz fitur ini sudah tersedia di SDK baru - https://docs.sentry.io/learn/filtering/?platform=javascript#before -breadcrumb

FYI: jangan lupa bahwa menggunakan SDK baru memberi Anda +50Kb untuk ukuran bundel #1552

@ 1999 itu menambahkan 12,7kB jika dibandingkan dengan raven-js, jangan gunakan ukuran non-gzip untuk perbandingan, karena itu tidak relevan.
Juga v5 (https://github.com/getsentry/sentry-javascript/pull/1919) akan menjadi <15kB, jadi Anda tidak perlu khawatir tentang peningkatan muatan Anda.

@kamilogorek Saya agak takut dengan PR dengan lebih dari 200 file perubahan, ini sepertinya lebih seperti SDK yang sama sekali baru daripada versi baru. Juga, apa tanggal rilis ETA itu?