Conversation
Updated the sanitizer function to improve input sanitization by removing style tags, normalizing pronouns, and increasing the maximum length limit.
Author
Changelog- Enhance input sanitization and normalize pronouns @AkhilRB0204
- Fix: duplicate healthcheck key in docker-compose db service [#462] @amahuli03
- fix: changed donate button link to direct to balancer github page [#465] @amahuli03
- Fix 401 by using adminApi instead of raw axios [#468] @amahuli03
- Fixed error 1 (openAI title sanitization) and added unit tests [#466] @amahuli03
- Fix: File downloads and opens used wrong API URL [#471] @amahuli03
- update site links on README [#479] @amahuli03
- Generate API docs [#472] @amahuli03
- Update files that Git should ignore [#457] @sahilds1
- fix: openAI fallback crash in title generation [#474] @amahuli03
- refactor: file upload uses font size and more lenient regex to extract titles [#475] @amahuli03
- [#441] [IMPROVE] Preload embedding model at startup [#461] @sahilds1
- [#187] [ADD] Documentation [#496] @sahilds1
- Add sandbox link to README usage section @sahilds1
- [#187] [IMPROVE] Documentation [#506] @sahilds1
- feat: integrate Gateway API, CORS config, and CNPG database for sandbox [#507] @TineoC
- [auth] fix auth problems permission boundaries [#489] @amahuli03
- [auth] account registration form and email activation [#493] @amahuli03
- [auth] Update login page, add token refresh, and fix password reset flow [#495] @amahuli03
- [#490] Improve AI Evals [#499] @sahilds1
- Update GitHub funding model configuration @sahilds1
- fix: add HTTP redirect route to ListenerSet for apex domain @TineoC |
…-healthchecks-docker-compose
…docker-compose Fix: duplicate healthcheck key in docker-compose db service
fix: changed donate button link to direct to balancer github page
Fix 401 by using adminApi instead of raw axios
…cular missed with default settings
… routed to the application instance
…sed, not where it's defined
Fixed error 1 (openAI title sanitization) and added unit tests
Fix: File downloads and opens used wrong API URL
Adds the balancer-db-credentials SealedSecret to the sandbox CNPG overlay so cnpg can set the balancer role password. The sealed secret uses the cluster's sealed-secrets controller cert and targets the cloudnative-pg namespace.
…ndbox-cluster The Database CR and SealedSecret should live in cfp-sandbox-cluster under the cloudnative-pg namespace, not in the app overlay.
[#187] [IMPROVE] Documentation
feat: integrate Gateway API, CORS config, and CNPG database for sandbox
[auth] fix auth problems permission boundaries
…tion [auth] account registration form and email activation
Refactor CORS_ALLOWED_ORIGINS to remove empty strings.
Refactor CORS_ALLOWED_ORIGINS to remove empty strings.
[auth] Update login page, add token refresh, and fix password reset flow
This PR refactors the assistant endpoint into smaller service modules (assistant orchestration + tool loop + prompts) and adds an evaluation script plus unit tests Changes: - Extracted assistant orchestration into assistant_services.py and tool-loop/retrieval logic into tool_services.py, updating the DRF view to call run_assistant. - Added an eval_assistant.py script (CSV output) and a small notebook for side-by-side response review. - Added focused unit tests for the new service modules and adjusted an existing uploadFile test import.
Removed placeholder username from GitHub funding model.
- Add balancer-http-redirect HTTPRoute attached to http sectionName in the ListenerSet, returning 301→HTTPS for balancerproject.org - Set disableNameSuffixHash to prevent ConfigMap hash suffix drift - Update ClusterIssuer annotation to letsencrypt-prod-gateway - Add overlay patches for both production and sandbox hostnames Refs: CodeForPhilly/cfp-live-cluster#160
fix: add HTTP redirect route to ListenerSet for apex domain
TineoC
approved these changes
Jul 14, 2026
TineoC
left a comment
TineoC
left a comment
Collaborator
There was a problem hiding this comment.
Merging to get fix onto main for hologit rebuild
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Improvements
Technical