#GitHub - CFC-Servers/cfc_err_forwarder: ...
lament hemlock
@past hazel Do you know why not all errors are forwarded? Such as those created manually with error() ?
Is there a specific requirement that must be met for the error to be forwarded?
past hazel
<@173634281116008448> Do you know why not all errors are forwarded? Such as thos...
They should be forwarding - we saw a bug today, though that may be causing errors with short stacks to fail.
Do you have a specific example?
Additionally, our rewrite is nearly merged. If you're not already, you could try using that branch, or wait until this evening when it's released?
lament hemlock
I can wait until this evening. But on my physgun server is appears only "engine" thrown errors are being forwarded. No manual error() is forward. Looking more now locally
calm kindle
new branch does catch error() for sure
lament hemlock
They should be forwarding - we saw a bug today, though that may be causing error...
example: lua_run error("Hello world")
past hazel
example: ``lua_run error("Hello world")``
Definitely the rewrite catches this.. But I'm also fairly sure that main should, too 😵💫
Not sure what's going on there
lament hemlock
Definitely the rewrite catches this.. But I'm also fairly sure that `main` shoul...
what exactly is the rewrite? lua_rewrite branch?
past hazel
what exactly is the rewrite? ``lua_rewrite`` branch?
Yep!
There are a couple of small bugs in there but I'll just go ahead and merge it now
✅
lament hemlock
will check it out today, thanks
calm kindle
its merged to master/main btw
lament hemlock
Yep!
it appears to be better but crc eats the error stack if u dont have a url set. Didn't use to be like this
eh i said that wrong.
THis used to just fail silently.
calm kindle
it appears to be better but crc eats the error stack if u dont have a url set. D...
wdym
crc?
you mean the client errors or something?
lament hemlock
you mean the client errors or something?
nah im just saying it should fail silently instead of spamming every lua refresh etc
calm kindle
sure i can just make it return early
lament hemlock
this is what my console looks like devving on the new version
devving locally, may just remove it from git but wasnt an issue before
calm kindle
i'll fix it
lament hemlock
awesome
calm kindle
this is what my console looks like devving on the new version
these errors seem to be different though, so if this still happens let me know
lament hemlock
these errors seem to be different though, so if this still happens let me know
Yeah this is just an error when no discord url is set
(no reason to set it while devving locally)
