search

near / fast-auth-signer

https://fast-auth-signer.vercel.app
MIT License
29 stars 8 forks source link

Upload Sentry Sourcemap #172

Closed hcho112 closed 4 months ago

hcho112 commented 5 months ago

This PR contains following changes:

  1. Upload DNS for fast-auth-signer-testnet (fast-auth-signer testnet)
  2. Upload souremaps for testnet and mainnet if SENTRY_AUTH_TOKEN is available during the build

Background: We have too many errors that hard to track where the error occurred from precisely. (Because we wrap around try, catch and put too many async within it) Aim of this PR is to upload source map during the build and upload to sentry so that it will reveal trans-piled code and reveal exactly where the error occurred.

Currently I have setup SENTRY_AUTH_TOKEN on mainnet environment

171

vercel[bot] commented 5 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fast-auth-signer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 12, 2024 2:01am
hcho112 commented 5 months ago

My apologies, This PR is still WIP. Created this PR to do further testing. Will update status once it is finally ready

hcho112 commented 5 months ago

image So I manage to get the source code revealing on sentry. Basically the reason why it wasn't showing was because I was trying to invoke this in localhost environment, but sentry server can't access to localhost on my machine.

I believe this is kind of ready, but at the same time, I haven't really had chance to test against github workflow side, so may need to pay attention after merge.

So at this point, I believe this PR is ready