Skip to content

Feature/docker cleanup#40

Merged
yassinsws merged 23 commits intomainfrom
feature/docker-cleanup
Jul 20, 2025
Merged

Feature/docker cleanup#40
yassinsws merged 23 commits intomainfrom
feature/docker-cleanup

Conversation

@yassinsws
Copy link
Collaborator

No description provided.

@waleedbaroudi waleedbaroudi force-pushed the feature/docker-cleanup branch from c4d1e0f to b9e3c8b Compare July 18, 2025 23:13
@yassinsws yassinsws force-pushed the main branch 2 times, most recently from 5eef348 to e958b9d Compare July 19, 2025 14:59
@waleedbaroudi waleedbaroudi force-pushed the feature/docker-cleanup branch 2 times, most recently from 90b0b4f to b92560d Compare July 19, 2025 17:58
@waleedbaroudi waleedbaroudi self-assigned this Jul 20, 2025
yassinsws and others added 21 commits July 20, 2025 14:18
- Changed Weaviate port from 8083 to 8087 to avoid conflict with auth service
- Updated WEAVIATE_PORT environment variable to match new port
- Modified deployment workflow to use helm uninstall + install instead of upgrade
- This resolves 'Duplicate value: http' and readiness probe conflicts
- Forces clean deployment to avoid resource conflicts from previous deploys
- Created JwtAuthenticationFilter for auth service
- Updated SecurityConfig to include JWT filter in chain
- Enables proper JWT token validation for authenticated endpoints like /api/auth/me
@waleedbaroudi waleedbaroudi force-pushed the feature/docker-cleanup branch from d2f89bb to 89550db Compare July 20, 2025 12:27
# Conflicts:
#	.github/workflows/studymate-rancher-cicd.yml.disabled
#	infra/helm/values.yaml
@yassinsws yassinsws merged commit 71257dd into main Jul 20, 2025
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants