 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>
		
	
		
			
				
	
	
		
			121 lines
		
	
	
		
			3.4 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			121 lines
		
	
	
		
			3.4 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright The OpenTelemetry Authors
 | |
| //
 | |
| // Licensed under the Apache License, Version 2.0 (the "License");
 | |
| // you may not use this file except in compliance with the License.
 | |
| // You may obtain a copy of the License at
 | |
| //
 | |
| //	http://www.apache.org/licenses/LICENSE-2.0
 | |
| //
 | |
| // Unless required by applicable law or agreed to in writing, software
 | |
| // distributed under the License is distributed on an "AS IS" BASIS,
 | |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 | |
| // See the License for the specific language governing permissions and
 | |
| // limitations under the License.
 | |
| 
 | |
| package resource // import "go.opentelemetry.io/otel/sdk/resource"
 | |
| 
 | |
| import (
 | |
| 	"context"
 | |
| 	"errors"
 | |
| 	"strings"
 | |
| 
 | |
| 	semconv "go.opentelemetry.io/otel/semconv/v1.24.0"
 | |
| )
 | |
| 
 | |
| type hostIDProvider func() (string, error)
 | |
| 
 | |
| var defaultHostIDProvider hostIDProvider = platformHostIDReader.read
 | |
| 
 | |
| var hostID = defaultHostIDProvider
 | |
| 
 | |
| type hostIDReader interface {
 | |
| 	read() (string, error)
 | |
| }
 | |
| 
 | |
| type fileReader func(string) (string, error)
 | |
| 
 | |
| type commandExecutor func(string, ...string) (string, error)
 | |
| 
 | |
| // hostIDReaderBSD implements hostIDReader.
 | |
| type hostIDReaderBSD struct {
 | |
| 	execCommand commandExecutor
 | |
| 	readFile    fileReader
 | |
| }
 | |
| 
 | |
| // read attempts to read the machine-id from /etc/hostid. If not found it will
 | |
| // execute `kenv -q smbios.system.uuid`. If neither location yields an id an
 | |
| // error will be returned.
 | |
| func (r *hostIDReaderBSD) read() (string, error) {
 | |
| 	if result, err := r.readFile("/etc/hostid"); err == nil {
 | |
| 		return strings.TrimSpace(result), nil
 | |
| 	}
 | |
| 
 | |
| 	if result, err := r.execCommand("kenv", "-q", "smbios.system.uuid"); err == nil {
 | |
| 		return strings.TrimSpace(result), nil
 | |
| 	}
 | |
| 
 | |
| 	return "", errors.New("host id not found in: /etc/hostid or kenv")
 | |
| }
 | |
| 
 | |
| // hostIDReaderDarwin implements hostIDReader.
 | |
| type hostIDReaderDarwin struct {
 | |
| 	execCommand commandExecutor
 | |
| }
 | |
| 
 | |
| // read executes `ioreg -rd1 -c "IOPlatformExpertDevice"` and parses host id
 | |
| // from the IOPlatformUUID line. If the command fails or the uuid cannot be
 | |
| // parsed an error will be returned.
 | |
| func (r *hostIDReaderDarwin) read() (string, error) {
 | |
| 	result, err := r.execCommand("ioreg", "-rd1", "-c", "IOPlatformExpertDevice")
 | |
| 	if err != nil {
 | |
| 		return "", err
 | |
| 	}
 | |
| 
 | |
| 	lines := strings.Split(result, "\n")
 | |
| 	for _, line := range lines {
 | |
| 		if strings.Contains(line, "IOPlatformUUID") {
 | |
| 			parts := strings.Split(line, " = ")
 | |
| 			if len(parts) == 2 {
 | |
| 				return strings.Trim(parts[1], "\""), nil
 | |
| 			}
 | |
| 			break
 | |
| 		}
 | |
| 	}
 | |
| 
 | |
| 	return "", errors.New("could not parse IOPlatformUUID")
 | |
| }
 | |
| 
 | |
| type hostIDReaderLinux struct {
 | |
| 	readFile fileReader
 | |
| }
 | |
| 
 | |
| // read attempts to read the machine-id from /etc/machine-id followed by
 | |
| // /var/lib/dbus/machine-id. If neither location yields an ID an error will
 | |
| // be returned.
 | |
| func (r *hostIDReaderLinux) read() (string, error) {
 | |
| 	if result, err := r.readFile("/etc/machine-id"); err == nil {
 | |
| 		return strings.TrimSpace(result), nil
 | |
| 	}
 | |
| 
 | |
| 	if result, err := r.readFile("/var/lib/dbus/machine-id"); err == nil {
 | |
| 		return strings.TrimSpace(result), nil
 | |
| 	}
 | |
| 
 | |
| 	return "", errors.New("host id not found in: /etc/machine-id or /var/lib/dbus/machine-id")
 | |
| }
 | |
| 
 | |
| type hostIDDetector struct{}
 | |
| 
 | |
| // Detect returns a *Resource containing the platform specific host id.
 | |
| func (hostIDDetector) Detect(ctx context.Context) (*Resource, error) {
 | |
| 	hostID, err := hostID()
 | |
| 	if err != nil {
 | |
| 		return nil, err
 | |
| 	}
 | |
| 
 | |
| 	return NewWithAttributes(
 | |
| 		semconv.SchemaURL,
 | |
| 		semconv.HostID(hostID),
 | |
| 	), nil
 | |
| }
 |