seq-tickets icon indicating copy to clipboard operation
seq-tickets copied to clipboard

Improvements around frontend errors & documentation regarding WAF rules

Open codymullins opened this issue 1 year ago • 2 comments

This issue tiptoes the line between bug, enhancement, and general information.

It appears Cloudflare will block some functionality based on rules detecting SQL in the querystring -- this is used in the Show Trace functionality.

Some thoughts:

  1. Add a section to the documentation (we're deploying with Azure + Cloudflare) that notes this could be blocked by WAF rules.
  2. Perhaps a POST in this case, with the data in the body, would get around it? I'm not totally sure.
  3. Although it was obvious as soon as I opened the network tab, I initially thought the error was during ingestion. No errors were in the logs - perhaps a way to track front-end errors here and surface them so they're apparent would be nice.

image

Additionally, the below error message was unhelpful.

How do I check the index is up to date? No idea. I spent too much time thinking this was an ingestion issue or some corrupted state.

Now, it makes sense. Seq couldn't find the trace. Some message improvements here -- "here's instructions on how to check the index", or "the request was blocked by your firewall/adblocker/yada yada"

image

codymullins avatar Oct 24 '24 18:10 codymullins

Thanks for the heads-up, Cody; we'll have to give this one some thought 👍

nblumhardt avatar Oct 24 '24 21:10 nblumhardt

Thanks - not expecting any quick resolution - we have a workaround for the ruleset for now. Hope the feedback helps.

codymullins avatar Oct 24 '24 23:10 codymullins

We've changed the message to:

If trace loading regularly times out, this
may indicate an indexing problem, or network infrastructure may be
blocking these requests.

Thanks, Cody!

nblumhardt avatar Nov 13 '24 05:11 nblumhardt