fix(issues): Don't treat %s as unique

[mrduncan]

If an event already has a parameterized logentry.message, ensure that uniq_id doesn't try to treat the %s as unique.

Example event payload:

{
  "logentry": {
    "message": "Hard time limit (%ss) exceeded for %s[%s]",
    "formatted": "Hard time limit (123s) exceeded for sentry.tasks.example[01234567-890a-bcde-f012-34567890abcd]",
    "params": [123, "sentry.tasks.example", "01234567-890a-bcde-f012-34567890abcd"]
  }
}

Prior to this change, these would have been parameterized as Hard time limit (%ss) exceeded for <uniq_id> (internal example).

This is currently pretty hard to test end-to-end and is also only catching a subset of possible % replacements, will follow up with a future PR to improve that but for now this will address the most common case.

Reminder: this parameterization is still only enabled experimentally and is not widely released.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.12%. Comparing base (9de90e9) to head (e7e21d0).
Report is 728 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master   #75090       +/-   ##
===========================================
+ Coverage   57.05%   78.12%   +21.07%     
===========================================
  Files        6735     6748       +13     
  Lines      300698   301174      +476     
  Branches    51732    51810       +78     
===========================================
+ Hits       171551   235301    +63750     
+ Misses     124394    59542    -64852     
- Partials     4753     6331     +1578     

Files	Coverage Δ
src/sentry/grouping/parameterization.py	96.52% <100.00%> (+20.41%)	⬆️

... and 2077 files with indirect coverage changes

[lobsterkatie]

Is there a reason not to add %d in here?

[mrduncan]

Mostly a combination of the difficulty in testing this and I want to limit the amount of additional work we have do do on each token_str until we have a better way to do better performance testing as well

[getsantry]

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you add the label WIP, I will leave it alone unless WIP is removed ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀