 131868bdca
			
		
	
	131868bdca
	
	
	
		
			
			Major security, observability, and configuration improvements:
## Security Hardening
- Implemented configurable CORS (no more wildcards)
- Added comprehensive auth middleware for admin endpoints
- Enhanced webhook HMAC validation
- Added input validation and rate limiting
- Security headers and CSP policies
## Configuration Management
- Made N8N webhook URL configurable (WHOOSH_N8N_BASE_URL)
- Replaced all hardcoded endpoints with environment variables
- Added feature flags for LLM vs heuristic composition
- Gitea fetch hardening with EAGER_FILTER and FULL_RESCAN options
## API Completeness
- Implemented GetCouncilComposition function
- Added GET /api/v1/councils/{id} endpoint
- Council artifacts API (POST/GET /api/v1/councils/{id}/artifacts)
- /admin/health/details endpoint with component status
- Database lookup for repository URLs (no hardcoded fallbacks)
## Observability & Performance
- Added OpenTelemetry distributed tracing with goal/pulse correlation
- Performance optimization database indexes
- Comprehensive health monitoring
- Enhanced logging and error handling
## Infrastructure
- Production-ready P2P discovery (replaces mock implementation)
- Removed unused Redis configuration
- Enhanced Docker Swarm integration
- Added migration files for performance indexes
## Code Quality
- Comprehensive input validation
- Graceful error handling and failsafe fallbacks
- Backwards compatibility maintained
- Following security best practices
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
		
	
		
			
				
	
	
		
			77 lines
		
	
	
		
			1.6 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			77 lines
		
	
	
		
			1.6 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| package pq
 | |
| 
 | |
| import (
 | |
| 	"fmt"
 | |
| 	"net"
 | |
| 	nurl "net/url"
 | |
| 	"sort"
 | |
| 	"strings"
 | |
| )
 | |
| 
 | |
| // ParseURL no longer needs to be used by clients of this library since supplying a URL as a
 | |
| // connection string to sql.Open() is now supported:
 | |
| //
 | |
| //	sql.Open("postgres", "postgres://bob:secret@1.2.3.4:5432/mydb?sslmode=verify-full")
 | |
| //
 | |
| // It remains exported here for backwards-compatibility.
 | |
| //
 | |
| // ParseURL converts a url to a connection string for driver.Open.
 | |
| // Example:
 | |
| //
 | |
| //	"postgres://bob:secret@1.2.3.4:5432/mydb?sslmode=verify-full"
 | |
| //
 | |
| // converts to:
 | |
| //
 | |
| //	"user=bob password=secret host=1.2.3.4 port=5432 dbname=mydb sslmode=verify-full"
 | |
| //
 | |
| // A minimal example:
 | |
| //
 | |
| //	"postgres://"
 | |
| //
 | |
| // This will be blank, causing driver.Open to use all of the defaults
 | |
| func ParseURL(url string) (string, error) {
 | |
| 	u, err := nurl.Parse(url)
 | |
| 	if err != nil {
 | |
| 		return "", err
 | |
| 	}
 | |
| 
 | |
| 	if u.Scheme != "postgres" && u.Scheme != "postgresql" {
 | |
| 		return "", fmt.Errorf("invalid connection protocol: %s", u.Scheme)
 | |
| 	}
 | |
| 
 | |
| 	var kvs []string
 | |
| 	escaper := strings.NewReplacer(`'`, `\'`, `\`, `\\`)
 | |
| 	accrue := func(k, v string) {
 | |
| 		if v != "" {
 | |
| 			kvs = append(kvs, k+"='"+escaper.Replace(v)+"'")
 | |
| 		}
 | |
| 	}
 | |
| 
 | |
| 	if u.User != nil {
 | |
| 		v := u.User.Username()
 | |
| 		accrue("user", v)
 | |
| 
 | |
| 		v, _ = u.User.Password()
 | |
| 		accrue("password", v)
 | |
| 	}
 | |
| 
 | |
| 	if host, port, err := net.SplitHostPort(u.Host); err != nil {
 | |
| 		accrue("host", u.Host)
 | |
| 	} else {
 | |
| 		accrue("host", host)
 | |
| 		accrue("port", port)
 | |
| 	}
 | |
| 
 | |
| 	if u.Path != "" {
 | |
| 		accrue("dbname", u.Path[1:])
 | |
| 	}
 | |
| 
 | |
| 	q := u.Query()
 | |
| 	for k := range q {
 | |
| 		accrue(k, q.Get(k))
 | |
| 	}
 | |
| 
 | |
| 	sort.Strings(kvs) // Makes testing easier (not a performance concern)
 | |
| 	return strings.Join(kvs, " "), nil
 | |
| }
 |