Some improvement for lockdown performance #2048
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add stub for authToSearch Applies to OpenUserJS#1548
Martii
added
enhancement
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Feb 24, 2025
* Got in too much of rush towards the end and boofed this. ;) Post OpenUserJS#2048
Martii
added a commit
that referenced
this pull request
Feb 24, 2025
* Got in too much of rush towards the end and boofed this. ;) Post #2048
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Feb 25, 2025
* Fix UI display missing libraries. Post OpenUserJS#2048 NOTE: Thank you for whoever you are pointing this out in a very subtle manner.
Martii
added a commit
that referenced
this pull request
Feb 25, 2025
* Fix UI display missing libraries. Post #2048 NOTE: Thank you for whoever you are pointing this out in a very subtle manner.
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 1, 2025
* Allow individual query auth's per public model * Bug fix on Sync as that's always needed with auth Post OpenUserJS#2048
Martii
added a commit
that referenced
this pull request
Mar 1, 2025
* Allow individual query auth's per public model * Bug fix on Sync as that's always needed with auth Post #2048
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 1, 2025
* More in-depth querying when available Post OpenUserJS#2048
Martii
added a commit
that referenced
this pull request
Mar 1, 2025
* More in-depth querying when available Post #2048
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 1, 2025
* Limit more to non-auth Post OpenUserJS#2048
Martii
added a commit
that referenced
this pull request
Mar 1, 2025
* Limit more to non-auth * Missed typo (style) Post #2048
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 2, 2025
* Trim up some of the data a bit who don't know how to use SEO properly. i.e. possible spamming/poisoning * Bug fix using incorrect identifier. Post OpenUserJS#2048 NOTE(s): * I tried for HOURS to get these settings into settings.json *(hence why the require is left in one file)* however RegExp just wouldn't have it no matter what I tried. Existing scripts worked, new scripts did not. If anyone has an idea PRs are welcome to move its location to settings.json. * DB migration will happen when I get around to it as this is very complicated since script source has to be reread but at least new scripts will have it.
Martii
added a commit
that referenced
this pull request
Mar 2, 2025
* Trim up some of the data a bit who don't know how to use SEO properly. i.e. possible spamming/poisoning * Bug fix using incorrect identifier. Post #2048 NOTE(s): * I tried for HOURS to get these settings into settings.json *(hence why the require is left in one file)* however RegExp just wouldn't have it no matter what I tried. Existing scripts worked, new scripts did not. If anyone has an idea PRs are welcome to move its location to settings.json. * DB migration will happen when I get around to it as this is very complicated since script source has to be reread but at least new scripts will have it.
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 2, 2025
* Revert prior fix as it broke things instead of clearing up identifier naming. * Change also how the horizontal ellipsis shows up. Post OpenUserJS#2056 OpenUserJS#2048 * We'll try this again.
Martii
added a commit
that referenced
this pull request
Mar 2, 2025
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 4, 2025
* We'll see if this improves things on pro. Dev is not useful as all queries finish in under 2000ms usually due to higher CPU/MEM configuration. * Rearrange indexing to closely match query order Post OpenUserJS#2048
Martii
added a commit
that referenced
this pull request
Mar 4, 2025
* We'll see if this improves things on pro. Dev is not useful as all queries finish in under 2000ms usually due to higher CPU/MEM configuration. * Rearrange indexing to closely match query order Post #2048
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 4, 2025
This reverts commit 4463468. Not the issue... reversion of OpenUserJS#2060 and post OpenUserJS#2048
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 4, 2025
* Since we've been running our own DB server we have way more connections... utilize them. * Re-Rearrange indexing to closely match query order and style conformance with spacing Post OpenUserJS#2048 NOTE: There's more to do...
Merged
Martii
added a commit
that referenced
this pull request
Mar 4, 2025
Abstract poolSize * Since we've been running our own DB server we have way more connections... utilize them. * Re-Rearrange indexing to closely match query order and style conformance with spacing Post #2048 NOTE: There's more to do...
Martii
added a commit
to Martii/OpenUserJS.org
that referenced
this pull request
Mar 4, 2025
* Keep lockdown minimal Post OpenUserJS#2048
Martii
added a commit
that referenced
this pull request
Mar 4, 2025
* Keep lockdown minimal Post #2048
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Applies to #1548