Skip to content

REST API and gRPC servers with JWT authentication and Swagger documentation - #6

Merged
RD2W merged 8 commits into
devfrom
feature/rest-api-study
Nov 12, 2025
Merged

REST API and gRPC servers with JWT authentication and Swagger documentation#6
RD2W merged 8 commits into
devfrom
feature/rest-api-study

Conversation

@RD2W

@RD2W RD2W commented Nov 9, 2025

Copy link
Copy Markdown
Owner

This PR introduces significant improvements to the go-notes project, including the addition of REST and gRPC servers, JWT authentication, Swagger documentation, and comprehensive testing.

Key Changes:

  • Added REST and gRPC servers with JWT authentication support
  • Implemented Swagger documentation for the API
  • Added tests for handlers, middleware, models, and repositories
  • Implemented graceful shutdown for the web server
  • Improved data storage structure (replaced timestamp-based JSON file names)
  • Updated GitHub Actions workflow (golangci-lint-action to v9)
  • Fixed README examples for correct API usage

These changes significantly enhance the project's functionality, reliability, and documentation.

RD2W added 8 commits November 9, 2025 19:49
This commit introduces new server implementations and removes obsolete components that were previously used for test data generation:

- Added gRPC server and client implementations with full CRUD operations
- Added web server with REST API endpoints and Gin framework
- Implemented JWT-based authentication and authorization middleware
- Added Swagger documentation for REST API endpoints
- Removed obsolete internal/service package (used for test data generation)
- Removed obsolete internal/logger package (used for monitoring test data)

The new servers provide complete API functionality, making the old test data generation components redundant.
- Change route to add the new Note from /api/notes/ to /api/notes
- Added clarification that curly braces are not part of JWT token
- Added debug logging to middleware and handlers
@RD2W
RD2W merged commit 31d3674 into dev Nov 12, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant