Skip to content
This repository was archived by the owner on Nov 23, 2025. It is now read-only.

Dev#3

Merged
RandithaK merged 8 commits intomainfrom
dev
Nov 8, 2025
Merged

Dev#3
RandithaK merged 8 commits intomainfrom
dev

Conversation

@RandithaK
Copy link
Member

No description provided.

RandithaK and others added 8 commits November 5, 2025 21:32
- Complete notification microservice implementation
- Email notification service with SMTP configuration
- Push notification support
- Subscription management
- Template system for notifications
- Data seeder with sample notifications
- OpenAPI/Swagger documentation
- Comprehensive DTOs and entities
- Full CRUD operations for notifications
- Email configuration with SendGrid/SMTP support
feat: Enhance role handling in GatewayHeaderFilter to include SUPER_ADMIN as ADMIN
feat: Add GitHub Actions workflows for building, packaging, and deploying Notification Service to Kubernetes
@gitguardian
Copy link

gitguardian bot commented Nov 8, 2025

⚠️ GitGuardian has uncovered 2 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
22222399 Triggered SMTP credentials 3e6908b notification-service/src/main/resources/application.properties View secret
22222401 Triggered SMTP credentials 3e6908b notification-service/src/main/resources/application-dev.properties View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@RandithaK RandithaK merged commit a70f7ae into main Nov 8, 2025
2 of 3 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Nov 8, 2025

Warning

Rate limit exceeded

@RandithaK has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 25 minutes and 52 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 36d8ed6 and 40845c9.

📒 Files selected for processing (39)
  • .github/workflows/build.yaml (1 hunks)
  • .github/workflows/deploy.yaml (1 hunks)
  • Dockerfile (1 hunks)
  • EMAIL_CONFIGURATION.md (1 hunks)
  • EMAIL_FIX_SUMMARY.md (1 hunks)
  • notification-service/.gitattributes (1 hunks)
  • notification-service/.gitignore (1 hunks)
  • notification-service/.mvn/wrapper/maven-wrapper.properties (1 hunks)
  • notification-service/Dockerfile (1 hunks)
  • notification-service/mvnw (1 hunks)
  • notification-service/mvnw.cmd (1 hunks)
  • notification-service/pom.xml (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/NotificationServiceApplication.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/config/GatewayHeaderFilter.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/config/NotificationProperties.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/config/OpenApiConfig.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/config/SecurityConfig.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/controller/NotificationController.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/dto/request/MarkAsReadRequest.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/dto/request/SubscribeRequest.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/dto/request/UnsubscribeRequest.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/dto/response/ApiResponse.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/dto/response/NotificationResponse.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/dto/response/SubscriptionResponse.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/entity/Notification.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/entity/Subscription.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/repository/NotificationRepository.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/repository/SubscriptionRepository.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/seeder/DataSeeder.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/service/NotificationService.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/service/SubscriptionService.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/service/impl/NotificationServiceImpl.java (1 hunks)
  • notification-service/src/main/java/com/techtorque/notification_service/service/impl/SubscriptionServiceImpl.java (1 hunks)
  • notification-service/src/main/resources/application-dev.properties (1 hunks)
  • notification-service/src/main/resources/application-prod.properties (1 hunks)
  • notification-service/src/main/resources/application.properties (1 hunks)
  • notification-service/src/test/java/com/techtorque/notification_service/NotificationServiceApplicationTests.java (1 hunks)
  • notification-service/src/test/resources/application-test.properties (1 hunks)
  • test_email_config.sh (1 hunks)
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant