Sentry-javascript: Добавить текст к элементам в панировочных сухарях ui.click

Итак, это то, что я хотел бы сделать, но меня беспокоит раздувание нашей функции сериализации DOM до такой степени, что она начинает добавлять заметные накладные расходы для обработчиков кликов.

Но, я буду исследовать.

Одной из простых альтернатив может быть просто сделать

target = elem.outerHTML;

который выведет элемент, на который нажали, точно так, как есть. Вы потеряете древовидную структуру, но, по моему личному опыту, это не так помогает, как просмотр полного фактического элемента, который я привык видеть в своем коде все время и, следовательно, быстрее могу распознать.

Это также решило бы https://github.com/getsentry/raven-js/issues/576.

@soroushhakami — outerHTML но отказались от этого служебного метода.

Причина, по которой мы не выбрали outerHTML (как прокомментировал @mitsuhiko ):

• формат селектора CSS может отображать больше информации с меньшим количеством символов
• outerHTML может включать зашумленные временные данные, например
  
  
  
  • длинные строки data-reactid (в React 0.14 и более ранних версиях)
  
  
  • динамически генерируемые атрибуты title предназначенные для всплывающих подсказок
  
  
• мы получили бы много пустых элементов, в зависимости от того, как настроена цель addEventListener
• у нас были кнопки с одним и тем же классом/содержимым, которые контекстуализируются только через информацию о предках

В принципе, outerHTML кажется хорошим, когда вы показываете кучу гипотетических примеров, где это выглядит хорошо, но наш опыт работы с ним вживую в течение нескольких недель показал, что этого недостаточно.

Ах, я вижу, имеет смысл, спасибо за объяснение! 👍

Я добавил идентификационные теги ко всем своим кнопкам. Удивлен, что они не отображаются в панировочных сухарях. Даже если бы это был пользовательский тег, я был бы доволен им. Но ID-метка кажется идеальной из-за своей уникальности.

@nblasgen - если вы можете предоставить неудачный тестовый пример, я был бы рад посмотреть, что

Но сейчас предполагается, что код который это проверяет .

+1 за показ идентификатора или пользовательского тега, например data-sentry-id="Some text here"

Я пока закрою этот вопрос, так как мы знаем об этой недостающей части, и мы вернемся к тому, как эта штука работает в следующей основной версии.

Ссылка: https://github.com/getsentry/raven-js/issues/783
Ссылка: https://github.com/getsentry/raven-js/issues/576

Просто хотел добавить - я использую реагирующие и стилизованные компоненты, поэтому имена моих классов генерируются автоматически, было бы здорово иметь больше контекста (без необходимости вручную captureBreadcrumb ):

@kamilogorek есть новости по этому

Есть ли возможность тонкой настройки клика "сообщение"?

Может быть, в breadcrumbCallback можно передать больше данных, чтобы люди могли выбрать то, что им нужно, не отправляя их часовому по умолчанию? Может быть, даже целый элемент дома?

@TuxujPes это будет возможно в следующем основном выпуске, но не в этом, поскольку для этого требуются некоторые общедоступные изменения API, которые мы можем внести только в основные выпуски.

@kamilogorek звучит здорово. Есть какие-нибудь оценки даты основного релиза?

Где-то ближе к концу третьего квартала (извините за такой долгий ответ, был в отпуске).

+1 за показ идентификатора или пользовательского тега, например data-sentry-id="Some text here"

Они могли бы проанализировать, есть ли эта информация, и взять ее с собой. Если он существует, оставьте селекторы с кнопкой для получения дополнительной информации.

@hiagodotme @TuxujPes @adamreisnz эта функция уже доступна в новом SDK - https://docs.sentry.io/learn/filtering/?platform=javascript#before -breadcrumb

К вашему сведению: просто не забывайте, что использование нового SDK дает вам +50 КБ для размера пакета # 1552.

@ 1999 он добавляет 12,7 КБ по сравнению с raven-js, не используйте для сравнения размеры без сжатия, так как это не имеет значения.
Также v5 (https://github.com/getsentry/sentry-javascript/pull/1919) будет <15 КБ, поэтому вам не придется беспокоиться об увеличении полезной нагрузки.

@kamilogorek Я немного боюсь PR с более чем 200 файлами изменений, это больше похоже на совершенно новый SDK, чем на новую версию. Кроме того, какова дата выпуска ETA?