Skip to content
Snippets Groups Projects
Verified Commit 2231ba73 authored by Matt's avatar Matt
Browse files

fix: Lower-case all remaining non-timestamp fields when ordering audit data....

fix: Lower-case all remaining non-timestamp fields when ordering audit data. Update tests to expect the fact that the engines still don't match.

Not many people use this project, even fewer likely setup OIDC to identify users. Almost no-one probably uses the audit page, let alone the ordering of columns, let alone cares that empty/symbols aren't ordered when they switched from MySQL to Postges...
If they do, I'm sorry I didn't spend an extra hour on tweaking SQL to work the same on all three (but hey, at least it's not degrading audit-search performance to performe a load more casts/case statements to fake/replace data for ordering)... but the search is still there, so this can easily filter data rather than relying on hundreds of rows of audit sorting in a particular way that the thing you wants happens to be on the top (or bottom).
... Now I've wasted an extra minute getting caught up in this.
Well, I got distracted by https://cookieplmonster.github.io/2025/04/23/gta-san-andreas-win11-24h2-bug/ and the comments https://news.ycombinator.com/item?id=43772311, which then lead me back to https://randomascii.wordpress.com/2024/10/01/life-death-and-retirement/ and am now sad and thus decided to give up on going further with this.

Issue #562
parent e81fd5e2
Pipeline #6556 failed with stages
in 1 hour, 33 minutes, and 43 seconds
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment