Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Unused Import in authorization.go #64

Closed
Feyzanrs opened this issue May 17, 2024 · 0 comments
Closed

Remove Unused Import in authorization.go #64

Feyzanrs opened this issue May 17, 2024 · 0 comments
Assignees
Labels
Be refactor Improvements or restructuring of existing functionality

Comments

@Feyzanrs
Copy link
Member

Feyzanrs commented May 17, 2024

In the authorization.go file within the internal/application/presentation/port/command directory, we need to clean up the code by removing an unused import. The import statement for "context" and the protobuf-related code are no longer necessary as the function CheckAuth has been commented out. This change will help maintain cleaner and more maintainable code by eliminating unnecessary dependencies.

The new version of the code:

package application

// AuthorizationCommandPort is a port for Hexagonal Architecture Pattern.
// It is used to communicate with the application layer.
type AuthorizationCommandPort interface {
    // CheckAuth returns true if the user is authorized to perform the action.
    // CheckAuth(ctx context.Context, authRequest *pb.AuthRequest) (*pb.AuthResponse, error)
}

Additionally, please make sure to push all changes to the feature/hsm/58/auth-service branch.

@Feyzanrs Feyzanrs added the refactor Improvements or restructuring of existing functionality label May 18, 2024
@Feyzanrs Feyzanrs assigned Smnrgcl and ebrarkesici and unassigned Smnrgcl May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Be refactor Improvements or restructuring of existing functionality
Projects
None yet
Development

No branches or pull requests

4 participants