Bump @sentry/browser from 8.43.0 to 8.47.0 #189
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bumps [@sentry/browser](https://github.com/getsentry/sentry-javascript) from 8.43.0 to 8.47.0. - [Release notes](https://github.com/getsentry/sentry-javascript/releases) - [Changelog](https://github.com/getsentry/sentry-javascript/blob/8.47.0/CHANGELOG.md) - [Commits](getsentry/sentry-javascript@8.43.0...8.47.0) --- updated-dependencies: - dependency-name: "@sentry/browser" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
|
Aviator Changeset actions:
|
Current Aviator status
This PR was merged manually (without Aviator). Merging manually can negatively impact the performance of the queue. Consider using Aviator next time.
See the real-time status of this PR on the
Aviator webapp.
Use the Aviator Chrome Extension
to see the status of your PR within GitHub.
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Review changes with |
|
Please follow naming conventions! 😿 |
|
Click here to review and test in web IDE: |
|
Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Files
|
There was a problem hiding this comment.
Pominęliśmy przegląd tego pull requesta. Wygląda na to, że został utworzony przez bota (hej, dependabot[bot]!). Zakładamy, że wie, co robi!
Original comment in English
We have skipped reviewing this pull request. It seems to have been created by a bot (hey, dependabot[bot]!). We assume it knows what it's doing!
|
Here's the code health analysis summary for commits Analysis Summary
|
Przewodnik Recenzenta od SourceryTen PR aktualizuje pakiet Diagram klas dla zaktualizowanych funkcji przeglądarki SentryclassDiagram
class SentryBrowser {
+updateSpanName()
+captureConsoleIntegration()
+captureMessage()
+captureException()
}
class Replay {
+maskSrcdocIframe
+rrwebIntegration
}
class ANRMonitor {
+captureMultipleEvents
+normalizeDebugPaths()
}
class Feedback {
+sendFeedback()
+asyncFeedback()
}
SentryBrowser --> Replay
SentryBrowser --> ANRMonitor
SentryBrowser --> Feedback
note for SentryBrowser "Wersja 8.47.0"
note for Replay "Nowe funkcje maskowania"
note for ANRMonitor "Ulepszone zarządzanie ANR"
note for Feedback "Ulepszony system opinii"
Zmiany na poziomie plików
Wskazówki i poleceniaInterakcja z Sourcery
Dostosowywanie swojego doświadczeniaUzyskaj dostęp do swojego pulpitu, aby:
Uzyskiwanie pomocy
Original review guide in EnglishReviewer's Guide by SourceryThis PR bumps the Class diagram for updated Sentry browser featuresclassDiagram
class SentryBrowser {
+updateSpanName()
+captureConsoleIntegration()
+captureMessage()
+captureException()
}
class Replay {
+maskSrcdocIframe
+rrwebIntegration
}
class ANRMonitor {
+captureMultipleEvents
+normalizeDebugPaths()
}
class Feedback {
+sendFeedback()
+asyncFeedback()
}
SentryBrowser --> Replay
SentryBrowser --> ANRMonitor
SentryBrowser --> Feedback
note for SentryBrowser "Version 8.47.0"
note for Replay "New masking features"
note for ANRMonitor "Enhanced ANR handling"
note for Feedback "Improved feedback system"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
This pull request failed to merge: PR cannot be automatically rebased, please rebase manually to continue. After you have resolved the problem, you should remove the Additional debug info: A merge conflict occurred while queueing this pull request. Potential conflicting pull requests: Please resolve manually and requeue. If resolving manually does not work, you may need to wait for previous PRs in the queue to merge. |
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 Do not merge outdated PRsWonderful, this rule succeeded.Make sure PRs are almost up to date before merging
|
🚀 CodeThreat Security Scan Completed for HulajDusza-serwisHello Team, Great news! We've just completed a thorough security scan for HulajDusza-serwis, and here's what we found: ⏱ Quick Overview
🛠 Detailed Vulnerability AnalysisWe've identified vulnerabilities across the codebase. Here's a detailed look:
🔗 Software Composition Analysis (SCA) Insightspackage-lock.jsonSeverity Summary: Critical: 5 High: 16 Medium: 14 Low: 5
📈 Next Steps & Full ReportTo dive deeper, click here to view the full report. It's essential to review these findings and plan the necessary fixes. If any of the critical/high issues need more discussion, let's set up a quick meeting to strategize our next steps. 🔒 Security isn't just a feature; it's a responsibility. Let's keep our codebase rock solid! |
| @@ -1,2 +1,2 @@ | |||
| class StickyHeader{constructor(){this.stickyHeader=$("header"),this.stickyMenu=$("nav.menu"),this.logo=$(".header-left img")}makeSticky(){var t=this.stickyHeader.offset().top,s=$(window).scrollTop(),t=t<s?"sticky":"",t=(this.stickyHeader.addClass(t),this.stickyMenu.addClass(t),150-s/300*50),t=Math.max(t,100),t=(this.logo.css("width",t+"px"),10-s/300*5),t=Math.max(t,5);this.stickyHeader.css("padding",t+"px 20px")}init(){$(window).scroll(()=>this.makeSticky())}}export default()=>(new StickyHeader).init(); | |||
| class StickyHeader{constructor(){this.stickyHeader=$("header"),this.stickyMenu=$("nav.menu"),this.logo=$(".header-left img")}makeSticky(){var s=this.stickyHeader.offset().top,t=$(window).scrollTop(),s=(s<t?(this.stickyHeader.addClass("sticky"),this.stickyMenu.addClass("sticky")):(this.stickyHeader.removeClass("sticky"),this.stickyMenu.removeClass("sticky")),150-t/300*50),s=Math.max(s,100),s=(this.logo.css("width",s+"px"),10-t/300*5),s=Math.max(s,5);this.stickyHeader.css("padding",s+"px 20px")}init(){$(window).scroll(()=>this.makeSticky())}}export default()=>(new StickyHeader).init(); | |||
There was a problem hiding this comment.
'arrow function syntax (=>)' is only available in ES6 (use 'esversion: 6').
'class' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
'export' is only available in ES6 (use 'esversion: 6').
's' is already defined.
Expected an assignment or function call and instead saw an expression.
Missing '()' invoking a constructor.
Missing semicolon.
| @@ -1,2 +1,2 @@ | |||
| class StickyHeader{constructor(){this.stickyHeader=$("header"),this.stickyMenu=$("nav.menu"),this.logo=$(".header-left img")}makeSticky(){var t=this.stickyHeader.offset().top,s=$(window).scrollTop(),t=t<s?"sticky":"",t=(this.stickyHeader.addClass(t),this.stickyMenu.addClass(t),150-s/300*50),t=Math.max(t,100),t=(this.logo.css("width",t+"px"),10-s/300*5),t=Math.max(t,5);this.stickyHeader.css("padding",t+"px 20px")}init(){$(window).scroll(()=>this.makeSticky())}}export default()=>(new StickyHeader).init(); | |||
| class StickyHeader{constructor(){this.stickyHeader=$("header"),this.stickyMenu=$("nav.menu"),this.logo=$(".header-left img")}makeSticky(){var s=this.stickyHeader.offset().top,t=$(window).scrollTop(),s=(s<t?(this.stickyHeader.addClass("sticky"),this.stickyMenu.addClass("sticky")):(this.stickyHeader.removeClass("sticky"),this.stickyMenu.removeClass("sticky")),150-t/300*50),s=Math.max(s,100),s=(this.logo.css("width",s+"px"),10-t/300*5),s=Math.max(s,5);this.stickyHeader.css("padding",s+"px 20px")}init(){$(window).scroll(()=>this.makeSticky())}}export default()=>(new StickyHeader).init(); | |||
There was a problem hiding this comment.
The makeSticky method is called on every scroll event, which can be highly frequent. This frequent DOM manipulation and layout reflow due to recalculating offsets can lead to significant performance degradation, especially on less powerful devices or with complex page layouts.
Recommendation: Consider debouncing or throttling the scroll event handler to reduce the frequency of these operations. This can be achieved using lodash's _.throttle or _.debounce functions, or a custom implementation if you prefer to avoid additional dependencies.
| @@ -1,2 +1,2 @@ | |||
| class StickyHeader{constructor(){this.stickyHeader=$("header"),this.stickyMenu=$("nav.menu"),this.logo=$(".header-left img")}makeSticky(){var t=this.stickyHeader.offset().top,s=$(window).scrollTop(),t=t<s?"sticky":"",t=(this.stickyHeader.addClass(t),this.stickyMenu.addClass(t),150-s/300*50),t=Math.max(t,100),t=(this.logo.css("width",t+"px"),10-s/300*5),t=Math.max(t,5);this.stickyHeader.css("padding",t+"px 20px")}init(){$(window).scroll(()=>this.makeSticky())}}export default()=>(new StickyHeader).init(); | |||
| class StickyHeader{constructor(){this.stickyHeader=$("header"),this.stickyMenu=$("nav.menu"),this.logo=$(".header-left img")}makeSticky(){var s=this.stickyHeader.offset().top,t=$(window).scrollTop(),s=(s<t?(this.stickyHeader.addClass("sticky"),this.stickyMenu.addClass("sticky")):(this.stickyHeader.removeClass("sticky"),this.stickyMenu.removeClass("sticky")),150-t/300*50),s=Math.max(s,100),s=(this.logo.css("width",s+"px"),10-t/300*5),s=Math.max(s,5);this.stickyHeader.css("padding",s+"px 20px")}init(){$(window).scroll(()=>this.makeSticky())}}export default()=>(new StickyHeader).init(); | |||
There was a problem hiding this comment.
The variable s is reused multiple times within the makeSticky method for different purposes, which makes the code difficult to read and maintain. This practice also leads to errors as seen in the existing comments about multiple definitions.
Recommendation: Use separate, descriptively named variables for different values and calculations. This will not only resolve the errors but also improve the readability and maintainability of the code.
🚀 CodeThreat Security Scan Completed for HulajDusza-serwisHello Team, Great news! We've just completed a thorough security scan for HulajDusza-serwis, and here's what we found: ⏱ Quick Overview
🛠 Detailed Vulnerability AnalysisWe've identified vulnerabilities across the codebase. Here's a detailed look:
🔗 Software Composition Analysis (SCA) Insightspackage-lock.jsonSeverity Summary: Critical: 5 High: 16 Medium: 14 Low: 5
📈 Next Steps & Full ReportTo dive deeper, click here to view the full report. It's essential to review these findings and plan the necessary fixes. If any of the critical/high issues need more discussion, let's set up a quick meeting to strategize our next steps. 🔒 Security isn't just a feature; it's a responsibility. Let's keep our codebase rock solid! |
🚀 CodeThreat Security Scan Completed for HulajDusza-serwisHello Team, Great news! We've just completed a thorough security scan for HulajDusza-serwis, and here's what we found: ⏱ Quick Overview
🛠 Detailed Vulnerability AnalysisWe've identified vulnerabilities across the codebase. Here's a detailed look:
🔗 Software Composition Analysis (SCA) Insightspackage-lock.jsonSeverity Summary: Critical: 5 High: 16 Medium: 14 Low: 5
📈 Next Steps & Full ReportTo dive deeper, click here to view the full report. It's essential to review these findings and plan the necessary fixes. If any of the critical/high issues need more discussion, let's set up a quick meeting to strategize our next steps. 🔒 Security isn't just a feature; it's a responsibility. Let's keep our codebase rock solid! |
🚀 CodeThreat Security Scan Completed for HulajDusza-serwisHello Team, Great news! We've just completed a thorough security scan for HulajDusza-serwis, and here's what we found: ⏱ Quick Overview
🛠 Detailed Vulnerability AnalysisWe've identified vulnerabilities across the codebase. Here's a detailed look:
🔗 Software Composition Analysis (SCA) Insightspackage-lock.jsonSeverity Summary: Critical: 5 High: 16 Medium: 14 Low: 5
📈 Next Steps & Full ReportTo dive deeper, click here to view the full report. It's essential to review these findings and plan the necessary fixes. If any of the critical/high issues need more discussion, let's set up a quick meeting to strategize our next steps. 🔒 Security isn't just a feature; it's a responsibility. Let's keep our codebase rock solid! |
|
|
This pull request failed to merge: PR cannot be automatically rebased, please rebase manually to continue. After you have resolved the problem, you should remove the Additional debug info: A merge conflict occurred while queueing this pull request. Potential conflicting pull requests: Please resolve manually and requeue. If resolving manually does not work, you may need to wait for previous PRs in the queue to merge. |
🚀 CodeThreat Security Scan Completed for HulajDusza-serwisHello Team, Great news! We've just completed a thorough security scan for HulajDusza-serwis, and here's what we found: ⏱ Quick Overview
🛠 Detailed Vulnerability AnalysisWe've identified vulnerabilities across the codebase. Here's a detailed look:
🔗 Software Composition Analysis (SCA) Insightspackage-lock.jsonSeverity Summary: Critical: 5 High: 16 Medium: 14 Low: 5
📈 Next Steps & Full ReportTo dive deeper, click here to view the full report. It's essential to review these findings and plan the necessary fixes. If any of the critical/high issues need more discussion, let's set up a quick meeting to strategize our next steps. 🔒 Security isn't just a feature; it's a responsibility. Let's keep our codebase rock solid! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bumps @sentry/browser from 8.43.0 to 8.47.0.
Release notes
Sourced from
@sentry/browser's releases.... (truncated)
Changelog
Sourced from
@sentry/browser's changelog.... (truncated)
Commits
eac1a88release: 8.47.08a7b29emeta: Update changelog for 8.47.0 (#14777)8107512feat(v8/replay): Mask srcdoc iframe contents per default (#14779)b1d393eref(v8/nextjs): Fix typo in source maps deletion warning (#14776)37fbd75feat(v8/node): Do not overwrite prismadb.systemin newer Prisma versions (...8e0db86chore(docs): Add docs for how to publish a release for previous majors (#14740)9c53d48feat(v8/node/deps): Bump@prisma/instrumentationfrom 5.19.1 to 5.22.0 (#14755)3a7743cfeat(core): AddupdateSpanNamehelper function [v8] (#14736)e80eefeMerge branch 'release/8.46.0' into v85387fderelease: 8.46.0Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Podsumowanie przez Sourcery
Ulepszenia:
Original summary in English
Summary by Sourcery
Enhancements: