Conversation
Replace scattered authorization patterns with Spring Security method-level security: - Add shared-security module with UserPrincipal and HeaderAuthentication - Add HeaderAuthenticationFilter (MVC) and HeaderAuthenticationWebFilter (WebFlux) - Replace requireAdmin() with @PreAuthorize("hasRole('ADMIN')") in user-service - Replace requireMediumOrAdmin() with @PreAuthorize("hasAnyRole('MEDIUM', 'ADMIN')") in divination-service - Add AccessDeniedException handlers to all GlobalExceptionHandlers - Owner-based checks remain in service layer to avoid duplicate DB lookups 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Code Coverage Report
Files
|
…ences Replace generic referenceId/referenceType with domain-specific spreadId and interpretationId fields in notification schema. This provides clearer semantics for NEW_INTERPRETATION notifications. Changes: - Add V2 migration: add spread_id/interpretation_id, migrate data, drop old columns - Update Notification entity with new nullable fields - Update NotificationDto, remove ReferenceType enum - Update EventConsumer to populate both fields from Kafka event - Update all unit, integration, and E2E tests - Update CLAUDE.md documentation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
SpreadMapper was creating InterpretationDto objects inline without using InterpretationMapper, which caused fileUrl to always be null. Now it delegates to InterpretationMapper.toDto() which correctly includes the fileUrl field. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Test verifies that interpretations with file attachments include the correct fileUrl in the spread details endpoint response. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Test verifies that GET /spreads/{id} returns interpretations with
correct fileUrl after file upload through the gateway.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
All phases completed: - Phase 1: SpreadMapper refactored to use InterpretationMapper - Phase 2: Integration test added for fileUrl in spread details - Phase 3: E2E test added and verified - Phase 4: Documentation updated 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
butvinm
added a commit
that referenced
this pull request
Jan 12, 2026
butvinm
added a commit
that referenced
this pull request
Jan 12, 2026
* Revert last two commits to return to lab3 state Reverts: - a336d28 Refactor authorization, fix issues (#13) - 3a23bed Notification and file service (#5) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Update highload-config to lab3-compatible state Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Set config-server default-label to revert-to-lab3 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Add highload-config management instructions to CLAUDE.md Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
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
1 participant
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.
No description provided.