Readit Newsbrungarc commented on Moving off of TypeScript, 2.5M lines of code engineering.usemotion.com... · Posted by u/caliChander
.NET and C# are a great choice for web backend and CRUD apps.
brungarc commented on I gave up on self-hosted Sentry (2024) bugsink.com/blog/why-i-ga... · Posted by u/roywashere
I do have one major complaint though, in dotnet, the tracing/errors are always captured regardless of the sampling rate. So you end up with a lot more memory usage on high throughput/low memory services with no way to lower it.
There's a ticket now open to stop this, but it's still in progress.
Avoiding allocations when a transaction isn’t sampled should be pretty trivial. Only gotcha is that you’d still trace id propagation to tie errors together regardless of transactions. But tracing and transactions got decoupled a while ago so shouldn’t be a big problem.
I’ll leave a comment on ticket pointing to this post
brungarc commented on Uno: Create Beautiful Cross Platform .NET Apps Faster platform.uno/... · Posted by u/thunderbong
You're correct about that sentiment from .NET developers historically but things have changed. At Skype, a company that's been under MS for a decade, more than half the devs use Macs.
C# is still my favorite language and I haven't used Windows since 2017.
JetBrains Rider is my IDE of choice on a Mac. It was pretty rough during the early releases of it, but for years now it's working well enough.
Also .NET Core 1.0, 1.1 and 2.0 was a ride. project.json anyone?
Fastforward to .NET 5, 6, 7, 8.. it's just doing great x-plat life and innovation
brungarc commented on Should You, Could You AOT? sentry.engineering/blog/s... · Posted by u/brungarc
Shared experience making a library compatible with AOT specifically for ASP.NET Core
brungarc commented on Spotlight: Sentry for Development spotlightjs.com/... · Posted by u/coloneltcb
Hey David! Few questions: I'm slightly stuck with the Vite plugin
Currently getting this when trying to do an npm run dev:
[plugin externalize-deps] No known conditions for "./vite-plugin" entry in "@spotlightjs/sidecar" package
The plugin "externalize-deps" was triggered by this import
vite.config.ts:7:29:
7 │ import spotlightSidecar from '@spotlightjs/sidecar/vite-plugin';
Also, I note the PHP SDK has gotten support for Spotlight - are there plans to do the same for the .NET SDK too?
Thanks for asking, created a ticket for .NET support here: https://github.com/getsentry/sentry-dotnet/issues/2955
I'm actually looking forward to it, I wanted to try on NuGet Trends but stopped on the first hurdle: https://github.com/dotnet/nuget-trends/pull/247
brungarc commented on Launch HN: Highlight.io (YC W23) – Open-source, full stack web app monitoring · Posted by u/podoman
Congrats on being the only commercial company to actually sponsor rrweb[0] rather than just fork it and contribute absolutely nothing back (or in the case of Sentry - remove their copyright and violate their license).
Seeing as you're "open-source", why chose to fork and detach the project rather than contribute directly to it? With a detached fork, other users can't even compare your changes to the original and pull in fixes. If you truly believed in the spirit of open source, you'd believe in working together and giving back; not just taking advantage of a free lunch.
It feels like all these "open-source" companies are just closed source but with open-source as a marketing gimmick.
Sentry’s license change on the fork was a mistake. It has been fixed though: https://github.com/getsentry/rrweb/pull/92
The fork exists as a ‘buffer’ to get some changes (features or bug fixes) out without the need to couple with the npm release of rrweb itself. Sentry engineers have several PRs in upstream rrweb merged and the goal is to increase the upstream contributions and close the gap between our fork and upstream. We’re currently porting our changes from v1 to the v2 branch.
I believe Sentry has made financial contributions to rrweb but directly to a maintainer. I'll let others who know details to comment on this. I'm sure more contributions will be done, as this is very much in the interest of Sentry anyway.
Worth noting, Sentry has been making larger contributions to OSS every year, as the company grows:
2021: https://blog.sentry.io/we-just-gave-154-999-dollars-and-89-c... 2022: https://blog.sentry.io/we-just-gave-260-028-dollars-to-open-...
In addition to that, there are contributions to open source done in the form of code that is, open source, such as the symbolication service: https://github.com/getsentry/symbolicator and many others: https://github.com/getsentry/
brungarc commented on Show HN: We’re open-sourcing our session replay tool github.com/highlight/high... · Posted by u/podoman
Right now we just capture network metadata like URL, connection type (fetch, xhr, websocket, etc), status code, filesize, etc. We additionally filter this data at the edge (before anything gets stored) for any known unsafe values like credit card numbers or SSNs, in case they somehow get into the URL.
As of right now we do not capture actual network request headers or bodies.
In addition to the network call data we already attached, that does go through data PII scrubbing on the edges as you mentioned, we're discussing capturing request/response body. We even talked about it today: https://github.com/getsentry/sentry-javascript/issues/7103
Avoiding any PII from getting into the system is always top of mind. And we're considering this as an opt-in regardless of scrubbing on the edge ingestion service.
Absolutely no reason other than "C# icky" -- they ended up with a platform that is crazy fast (and fast scales way easier, it handles a dumb amount of traffic without a lot of crazy design)
Startup culture is toxic AF at times, bad engineering decisions for cargo cult stuff.